Skip to content

Commit

Permalink
[ModuloSchedule] Devirtualize PeelingModuloScheduleExpander::expand a…
Browse files Browse the repository at this point in the history
…s it's not needed

The use case is out of tree code deriving from this class - but without
a need to use the base class polymorphically, so skip the virtualization
and virtual dtor.

Post-commit review from 50ac7ce
  • Loading branch information
dwblaikie committed Jul 7, 2020
1 parent 10c82ee commit 7a99aab
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions llvm/include/llvm/CodeGen/ModuloSchedule.h
Expand Up @@ -282,9 +282,8 @@ class PeelingModuloScheduleExpander {
LiveIntervals *LIS)
: Schedule(S), MF(MF), ST(MF.getSubtarget()), MRI(MF.getRegInfo()),
TII(ST.getInstrInfo()), LIS(LIS) {}
virtual ~PeelingModuloScheduleExpander() {}

virtual void expand();
void expand();

/// Runs ModuloScheduleExpander and treats it as a golden input to validate
/// aspects of the code generated by PeelingModuloScheduleExpander.
Expand Down

0 comments on commit 7a99aab

Please sign in to comment.