[PATCH] D21699: [PM] Port PartialInlining to new PM

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 24 14:59:23 PDT 2016


davide accepted this revision.
davide added a comment.
This revision is now accepted and ready to land.

This patch is nice. Just one minor comment. After that, LGTM.


================
Comment at: lib/Transforms/IPO/PartialInlining.cpp:33
@@ +32,3 @@
+struct PartialInlinerLegacyPass : public ModulePass {
+  void getAnalysisUsage(AnalysisUsage &AU) const override {}
+  static char ID; // Pass identification, replacement for typeid
----------------
This one can be removed, no? 


http://reviews.llvm.org/D21699





More information about the llvm-commits mailing list