[PATCH] D156565: Diagnose use of VLAs in C++ by default

Tue Aug 29 05:52:13 PDT 2023

cor3ntin added a comment.

In D156565#4605027 <https://reviews.llvm.org/D156565#4605027>, @shafik wrote:

> In D156565#4599812 <https://reviews.llvm.org/D156565#4599812>, @aaron.ballman wrote:
>
>> Enable the diagnostic by default in C++ language modes, and under -Wall in GNU++ language modes.
>
> I am happy with that approach.

Ditto! Some of the GCC folks also expressed a desire for this direction on their side.
Given that, and how surprising VLAs can be to c++ folks, It seems reasonable to go ahead with this patch

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

https://reviews.llvm.org/D156565