[PATCH] D133108: [clang] Rework IsTailPaddedMemberArray into isFlexibleArrayMemberExpr
serge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 16 14:26:32 PDT 2022
serge-sans-paille updated this revision to Diff 460899.
serge-sans-paille edited the summary of this revision.
serge-sans-paille added a comment.
Address minor nits, plus add a test case that showcases the extra (legitimate!) warning we now get when accessing a "struct hack" field with an index that cannot be represented in associated address space.
Kudos to @aaron.ballman for spotting this one!
As a comparison, current behavior for the same test case: https://godbolt.org/z/86Wcbrz5q (only the flexible array member is flagged with a warning, the struct hacks are ignored)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133108/new/
https://reviews.llvm.org/D133108
Files:
clang/lib/Sema/SemaChecking.cpp
clang/test/Sema/unbounded-array-bounds.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D133108.460899.patch
Type: text/x-patch
Size: 6101 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220916/14e9fd38/attachment.bin>
More information about the cfe-commits
mailing list