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

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: include/llvm/LTO/LTO.h:389
+    /// Keep track if the symbol was referenced by the regular LTO partition.
+    bool VisibleToRegularLTOPartition = false;
+
----------------
mehdi_amini wrote:
> pcc wrote:
> > Don't you just need one flag: VisibleOutsideThinLTO (or something).
> I think one flag should be enough, but shouldn't we have the same flag for LTO internalization?
> 
Wouldn't that be redundant with Partition == 0?


https://reviews.llvm.org/D23488