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

Tue Dec 1 07:04:34 PST 2020

Xiangling_L added inline comments.

================
Comment at: clang/test/Layout/aix-bitfield-alignment.cpp:1
+// RUN: %clang_cc1 -triple powerpc-ibm-aix-xcoff -fdump-record-layouts \
+// RUN:     -fsyntax-only -faix-pragma-pack -x c++ %s | \
----------------
sfertile wrote:
> I suggest we split the test cases which are C++ only into a sperate file, and run the cases that are valid in both C and C++ as both to ensure there are no differences between C and C++ layout fir bit fields.
Good idea. I split the testcase as you suggested. But since C and C++ layout dumping are slightly different in format, the `CHECK`s are adjusted accordingly to avoid making testcase too messy. 

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

https://reviews.llvm.org/D87029