[PATCH] D20290: [ThinLTO] Refactor ODR resolution and internalization (NFC)

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Thu May 19 08:32:06 PDT 2016


tejohnson added inline comments.

================
Comment at: lib/LTO/LTO.cpp:90
@@ +89,3 @@
+      if (GlobalValue::isLocalLinkage(S->linkage()))
+        S->setLinkage(GlobalValue::ExternalLinkage);
+    } else {
----------------
joker.eph wrote:
> This does not sound correct to me.
This is needed so that it is not internalized again in thinLTOInternalizeModule (which runs after promotion).


http://reviews.llvm.org/D20290





More information about the llvm-commits mailing list