[PATCH] D19096: ModuleLinker: do not always pull-in linkonce_odr when performing ThinLTO importing

[Mehdi AMINI via llvm-commits]

joker.eph added inline comments.

================
Comment at: lib/LTO/ThinLTOCodeGenerator.cpp:160
@@ -153,3 +159,3 @@
     else
       return GlobalValue::AvailableExternallyLinkage;
     break;
----------------
(This code was removed)

================
Comment at: tools/llvm-link/llvm-link.cpp:280
@@ +279,3 @@
+    // Instruct the linker to not automatically import linkonce defintion.
+    unsigned Flags = Linker::Flags::DontForceLinkLinkonceODR;
+
----------------
tejohnson wrote:
> Doesn't the ODR resolution need to be done to set this flag? I don't see it being done in llvm-link.
I'm not sure what you'd see to resolve here, this is the mode where llvm-link is instructed on the command line to import a specific list of functions.


http://reviews.llvm.org/D19096