[PATCH] D32544: LTO: Mark undefined module inline asm symbols as visible outside of ThinLTO.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 12:20:20 PDT 2017


pcc added a comment.

In https://reviews.llvm.org/D32544#738497, @mehdi_amini wrote:

> Won't this disable *linker* dead-stripping?


I don't think so. LTO will only internalize a global if the linker tells it that is not a GC root. This change disables internalization for some entities that would otherwise be internalized, but linker dead stripping is not inhibited because even if the symbol had external linkage, it would still not be a GC root.

> I don't understand the motivation for this change right now?

See PR32798.


Repository:
  rL LLVM

https://reviews.llvm.org/D32544





More information about the llvm-commits mailing list