[PATCH] D20290: [ThinLTO] Change ODR resolution and internalization to be index-based

[Teresa Johnson via llvm-commits]

tejohnson added inline comments.

================
Comment at: lib/LTO/ThinLTOCodeGenerator.cpp:339
@@ -344,2 +338,3 @@
+  }
 }
 
----------------
joker.eph wrote:
> Yes in ld64 and regular monolithic LTO, we are internalizing linkonce_ODR if they are not exported.
> Can weak functions defined in the main binary be overriden at runtime?
> It may be a platform specific question...
> 
Agree it is going to be platform specific. So then I think that the new version of the patch is better, and each linker will need to handle appropriately (e.g. here by adding to preserveSymbol(), in gold by using the callback to query gold's resolution, and eventually with pcc's API via the resolution info provided to the API).

Do you want to retest this new version before I commit?


http://reviews.llvm.org/D20290