================ @@ -99,15 +99,15 @@ class InputChunk { // the beginning of the output section this chunk was assigned to. int32_t outSecOff = 0; - uint8_t sectionKind : 3; + uint32_t sectionKind : 3; ---------------- ojhunt wrote:
Unfortunately I can :( The MS ABI only packs bit-fields if they have the same storage size, e.g. ```cpp struct S { bool b: 1; int i: 1 }; ``` is 8 bytes. It's a large part of why we use preferred_type and unsigned bit-fields instead of just enum bit-fields. This PR is the continuing series of fixing the existing cases where the storage types are wrong (per MS :D), so we can then enable the warning for this behavior, so that we can then just start using enums without risking silently terrible padding with MSVC https://github.com/llvm/llvm-project/pull/140493 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits