[PATCH] D23488: ThinLTO: add early "dead-stripping" on the Index

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

In https://reviews.llvm.org/D23488#627136, @tejohnson wrote:

> In https://reviews.llvm.org/D23488#627135, @mehdi_amini wrote:
>
> > As an alternative, have you considered having a bit "used" (or "do not dead strip") that would express in the summary exactly what we want?
>
>
> Yes, that would be another alternative, I thought about that but was hoping it could be avoided, but maybe not easily. The bit would probably need to be set for any variable that starts with "llvm." to be safely conservative (e.g. we should also consider anything in llvm.used or llvm.compiler.used as potentially live).


llvm.used is definitely a must-be root, llvm.compiler.used are supposed to be able to be dead-stripped at link-time.


https://reviews.llvm.org/D23488