[PATCH] D19813: Move "Eliminate Available Externally" immediately after the inliner

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Tue May 3 06:40:11 PDT 2016


tejohnson added a subscriber: tejohnson.

================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:410
@@ +409,3 @@
+    // compiling an object file for later LTO. For LTO we want to preserve
+    // these so they are eligible for inlining at link-time. Note if they
+    // are unreferenced they will be removed by GlobalDCE later, so
----------------
Does the same concern apply to ThinLTO (want to preserve them for inlining later on as in LTO)? If not, why?


http://reviews.llvm.org/D19813





More information about the llvm-commits mailing list