[PATCH] D38154: [PassManager] Run global opts after the inliner

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 14:35:24 PDT 2017


efriedma added inline comments.


================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:513
+    PM.add(createGlobalOptimizerPass());
+    PM.add(createGlobalDCEPass());
   }
----------------
Are you sure this is where you meant to put these passes...?  GlobalOpt and GlobaDCE are module passes, so you're separating the inliner from the function simplification passes.


https://reviews.llvm.org/D38154





More information about the llvm-commits mailing list