[PATCH] D126864: [clang] Introduce -fstrict-flex-arrays=<n> for stricter handling of flexible arrays
serge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 29 05:49:19 PDT 2022
serge-sans-paille updated this revision to Diff 440942.
serge-sans-paille added a subscriber: chandlerc.
serge-sans-paille added a comment.
Code updated to take into account two situations:
- size resulting from macro expansion. Previous behavior was inconsistent in that situation. I chose to consider that `int a` and `int a[N]` where N is a macro definition both are FAM. That's GCC behavior too. CCing @chandlerc because he introduced that feature, probably because that could make an array a FAM just based on preprocessor flag (?)
- FAM within non-standard layout C++ object. GCC allows that behavior and clang used to have an inconsistent approach here. I'd like to stick to GCC behavior.
Note that we still diverge from GCC behavior for `int a[N]` when `N` results from template expansion. I could revert that too.
CHANGES SINCE LAST ACTION
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 25898 bytes
Desc: not available
More information about the cfe-commits