[PATCH] D29104: Add !associated metadata.
Evgeniy Stepanov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 24 17:25:20 PST 2017
eugenis added a comment.
In https://reviews.llvm.org/D29104#655651, @mehdi_amini wrote:
> I mean the specific optimizations that won't happen that motivate not using llvm.compiler.used.
I'm not sure. Perhaps, link time devirtualization? Without this, ASan would have to add every single global to llvm.compiler.used, which would make globaldce not do anything at all. If anything, it is bad for the object file size.
The real reason is w/o the GlobalDCE change !associated has strange semantics - it protects a global from linker GC, but only if said global survives compiler GC. I don't see how that is useful.
Repository:
rL LLVM
https://reviews.llvm.org/D29104
More information about the llvm-commits
mailing list