[PATCH] D103979: [lld-macho] Have dead-stripping work with literal sections

[Thorsten via Phabricator via llvm-commits]

tschuett added inline comments.


================
Comment at: lld/MachO/InputSection.h:123
+  bool live : 1;
+  uint32_t hash : 31;
   // Offset from the start of the containing output section.
----------------
craig.topper wrote:
> int3 wrote:
> > oontvoo wrote:
> > > IMHO, 31 is a bit weird. (It's only 1 bit difference!)
> > > 
> > 1 extra bit here makes the struct take up 1 extra word :)
> > 
> > This was taken from LLD-ELF; I have yet to profile it myself, but it seemed like a reasonable idea
> Should `live` also be uint32_t? If i recall, MSVC needs the types of bitfields to be the same in order to merge the storage.
Could you add a static_assert for proof and documentation?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103979/new/

https://reviews.llvm.org/D103979