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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
