[PATCH] D105660: [PowerPC][AIX] Add warning when alignment is incompatible with XL

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 9 06:29:29 PDT 2021


aaron.ballman added inline comments.


================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:3255-3256
+def warn_not_xl_compatible
+    : Warning<"requested alignment of arguments 16 bytes or greater is not"
+              " compatible with previous versions of the AIX XL compiler">,
+      InGroup<DiagGroup<"builtin-assume-aligned-alignment">>;
----------------
Should we be talking about the AIX XL compiler in a Clang diagnostic?


================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:3957
 
+  if (Context.getTargetInfo().getTriple().isOSAIX() && (AlignVal >= 16))
+    Diag(AttrLoc, diag::warn_not_xl_compatible) << E->getSourceRange();
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105660



More information about the cfe-commits mailing list