This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG66225db98d83: [PowerPC][AIX] Add warning when alignment is
incompatible with XL (authored by ZarkoCA).
Repository:
rG LLVM Github Monorepo
CHANGE
ZarkoCA 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
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, thank you!
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:3255-3256
+def warn_not_xl_compatible
+: Warning<"requested alignment of argum
ZarkoCA marked 2 inline comments as done.
ZarkoCA 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 wit
ZarkoCA updated this revision to Diff 358934.
ZarkoCA added a comment.
- Fix formatting in test case
- Reword warning message
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105660/new/
https://reviews.llvm.org/D105660
Files:
clang/include/clang/B
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 co
ZarkoCA marked an inline comment as done.
ZarkoCA 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 wit
ZarkoCA updated this revision to Diff 358760.
ZarkoCA added a comment.
- Add more RUN lines to test
- change `if` condition to be one line
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105660/new/
https://reviews.llvm.org/D105660
Files:
clang/in
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 co
ZarkoCA updated this revision to Diff 358261.
ZarkoCA added a comment.
- Add a warning group for this warning
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105660/new/
https://reviews.llvm.org/D105660
Files:
clang/include/clang/Basic/DiagnosticG
ZarkoCA marked an inline comment as done.
ZarkoCA 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 wit
ZarkoCA updated this revision to Diff 358087.
ZarkoCA added a comment.
- Warning should only apply to members of struct
- Re-word warning slightly
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105660/new/
https://reviews.llvm.org/D105660
Files:
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 co
ZarkoCA created this revision.
ZarkoCA added reviewers: sfertile, cebowleratibm.
ZarkoCA added a project: PowerPC.
Herald added subscribers: shchenz, nemanjai.
Herald added a reviewer: aaron.ballman.
ZarkoCA requested review of this revision.
Herald added a project: clang.
Herald added a subscriber
14 matches
Mail list logo
