[PATCH] D18986: [ThinLTO] Prevent importing of "llvm.used" values

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 18 13:30:54 PDT 2016


joker.eph added a comment.

Also, could we promote and rename while keeping a private alias with the old name for these globals?


================
Comment at: include/llvm/IR/Module.h:757
@@ -751,1 +756,3 @@
+                                           bool CompilerUsed);
+
 /// An raw_ostream inserter for modules.
----------------
tejohnson wrote:
> joker.eph wrote:
> > Why was this code-motion needed?
> So that it can be used in lib/Analysis/ModuleSummaryAnalysis.cpp
Oh and you can't introduce a dependency from `Analysis` to `TransformsUtil`


http://reviews.llvm.org/D18986





More information about the llvm-commits mailing list