[PATCH] D118350: [AIX][PowerPC] Emit byval alignment warning only when struct member is passed to a function

Zarko Todorovski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 27 04:21:50 PST 2022


ZarkoCA created this revision.
ZarkoCA added reviewers: hubert.reinterpretcast, sfertile, cebowleratibm, stevewan, Jake-Egan, aaron.ballman, tmatheson, dnsampaio.
ZarkoCA added projects: clang, PowerPC.
Herald added subscribers: shchenz, nemanjai.
ZarkoCA requested review of this revision.
Herald added a subscriber: cfe-commits.

Previous warning went on whenever a struct with a struct member with alignment => 16 was declared.
This led to too many false positives and led to diagnostic lit failures due to it being emitted
too frequently. Only emit the warning when such an argument is passed to a caller function since
this is where the potential binary compatibility issue with XL 16.1.0 and older exists.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D118350

Files:
  clang/include/clang/Sema/Sema.h
  clang/lib/Sema/SemaChecking.cpp
  clang/lib/Sema/SemaDeclAttr.cpp
  clang/test/Sema/aix-attr-align.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118350.403591.patch
Type: text/x-patch
Size: 6325 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220127/95c8e509/attachment.bin>


More information about the cfe-commits mailing list