Xiangling_L added inline comments.
================
Comment at: clang/lib/AST/RecordLayoutBuilder.cpp:1225
+ Context.getTargetInfo().getTriple().isPS4() ||
+ Context.getTargetInfo().getTriple().isOSAIX()))
+ ? CharUnits::One()
----------------
hubert.reinterpretcast wrote:
> Thanks; verified that this is correct with `xlclang++` from IBM XL C/C++ for
> AIX with:
> ```
> struct A {
> char x;
> };
> struct B {
> int x;
> };
> struct __attribute__((__packed__)) C : A, B {} c;
> ```
>
> Length is 5:
> ```
> [10] m 0x00000004 .bss 1 extern c
> [11] a4 0x00000005 0 0 CM RW 0 0
> ```
>
> @Xiangling_L, I suggest adding a case for this to the tests.
Sure, I will add it.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79719/new/
https://reviews.llvm.org/D79719
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
