[PATCH] D87029: [AIX] Implement AIX special bitfield related alignment rules

Jason Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 21 08:09:04 PDT 2020


jasonliu added a comment.

I think it would help the review if we could put the NFC portion(e.g. TypeSize -> StorageUnitSize) to a new patch, and give some rationale about the NFC change.



================
Comment at: clang/lib/AST/RecordLayoutBuilder.cpp:624
   /// a byte, but larger units are used if IsMsStruct.
   unsigned char UnfilledBitsInLastUnit;
+  /// LastBitfieldStorageUnitSize - If IsMsStruct, represents the size of the
----------------



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

https://reviews.llvm.org/D87029



More information about the cfe-commits mailing list