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

David Tenty via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 23 07:33:22 PST 2023


daltenty added inline comments.


================
Comment at: clang/test/SemaCXX/class-layout.cpp:616-617
 
+// This ends the #if !defined(__MVS__) && !defined(__AIX__) block from line 12 
+#endif
+
----------------
nit: line numbers may shift on you as other commits touch this file. Prefer  a same line comment that keeps the comment together with the actual directive.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142358



More information about the cfe-commits mailing list