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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
