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

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 22 09:30:28 PST 2016


tejohnson added a comment.

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

> I just realized I haven't addressed this yet. Looks like it requires adding a new flag into the SymbolResolution, since we can't currently distinguish between a value referenced between different ThinLTO partitions and the case where we have a reference between a ThinLTO partition and the regular LTO partition - in both cases the resulting GlobalResolution Partition will be External. I'll probably have to add a VisibleToRegularLTOPartition flag onto the SymbolResolution and set it when the Partition == External and we have seen a reference from Partition 0. I'll do that and add a test.


Correction: the new flag will go into the GlobalResolution not the SymbolResolution!


https://reviews.llvm.org/D23488





More information about the llvm-commits mailing list