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

[Craig Topper via Phabricator via llvm-commits]

craig.topper 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.
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103979