[PATCH] D28978: [ThinLTO] Add an auto-hide feature

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

Does this need to be in the summary? I would have thought that the linker can use `canBeOmittedFromSymbolTable()` to implement its own auto-hiding logic. That is what ELF lld currently does.



================
Comment at: llvm/lib/Bitcode/Writer/BitcodeWriter.cpp:980
   RawFlags = (RawFlags << 4) | Flags.Linkage; // 4 bits
+  RawFlags |= (Flags.AutoHide << 6);         // bool
 
----------------
tejohnson wrote:
> Move this up and << 2 (consistent with LiveRoot and other bool flag setup)
It looks like we are setting this variable in a very confusing way. I'd move the linkage first and then shift NotEligibleToImport and LiveRoot by 4 more.


https://reviews.llvm.org/D28978