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

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini 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;
+
----------------
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?



https://reviews.llvm.org/D23488