OSDN Git Service

[ModuloSchedule] Devirtualize PeelingModuloScheduleExpander::expand as it's not needed
authorDavid Blaikie <dblaikie@gmail.com>
Tue, 7 Jul 2020 01:01:57 +0000 (18:01 -0700)
committerDavid Blaikie <dblaikie@gmail.com>
Tue, 7 Jul 2020 01:05:32 +0000 (18:05 -0700)
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 50ac7ce94f34c5f43b02185ae0c33e150e78b044

llvm/include/llvm/CodeGen/ModuloSchedule.h

index be0108d..1aa2320 100644 (file)
@@ -282,9 +282,8 @@ public:
                                 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.