[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