[PATCH] D126864: [clang] Introduce -fstrict-flex-arrays=<n> for stricter handling of flexible arrays

Tue Aug 15 05:51:17 PDT 2023

aaron.ballman added a comment.

In D126864#4586307 <https://reviews.llvm.org/D126864#4586307>, @efriedma wrote:

> RecordDecl::hasFlexibleArrayMember() is supposed to reflect the standard's definition of a flexible array member, which only includes incomplete arrays.  The places that care about other array members use separate checks.  We wouldn't want to accidentally extend the non-standard treatment of arrays with fixed bound to other places.
>
> I think there was an effort to try to refactor uses of StrictFlexArrays to a centralized helper, but I don't remember what happened to it.

Are you thinking of https://reviews.llvm.org/D134791 perchance?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126864/new/

https://reviews.llvm.org/D126864