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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to