[PATCH] D142358: Opting out of Clang 15 ABI Changes for AIX and z/OS

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 23 20:43:04 PST 2023


hubert.reinterpretcast accepted this revision.
hubert.reinterpretcast added a comment.
This revision is now accepted and ready to land.

Thanks @nicolerabjohn; this LGTM!



================
Comment at: clang/test/SemaCXX/class-layout.cpp:685
+#if (defined(CLANG_ABI_COMPAT) && CLANG_ABI_COMPAT <= 15) || !defined(__MVS__)
 _Static_assert(_Alignof(t2) == 1, "");
+#else
----------------
Just a note: As mentioned above, for some releases of Open XL on z/OS, the `-fclang-abi-compat` level that gives this result is less than 15 (but, then again, the behaviour expected by this line persisted in vanilla Clang 15). What is more important is the default behaviour (confirmed below) anyway.


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

https://reviews.llvm.org/D142358



More information about the cfe-commits mailing list