[Bug c++/110848] Consider enabling -Wvla by default in C++ modes

2023-07-29 Thread aaron at aaronballman dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110848 --- Comment #6 from Aaron Ballman --- (In reply to Andrew Pinski from comment #4) > Maybe my issue is this has been a documented extension for 20 years now. Which is totally fair -- we don't usually enable congratulatory diagnostics by

[Bug c++/110848] Consider enabling -Wvla by default in C++ modes

2023-07-28 Thread muecker at gwdg dot de via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110848 Martin Uecker changed: What|Removed |Added CC||muecker at gwdg dot de --- Comment #5

[Bug c++/110848] Consider enabling -Wvla by default in C++ modes

2023-07-28 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110848 --- Comment #4 from Andrew Pinski --- Maybe my issue is this has been a documented extension for 20 years now. -pedantic or -std=c++NN has always rejected it like it should. GCC has other extensions which folks could use by accident too (like

[Bug c++/110848] Consider enabling -Wvla by default in C++ modes

2023-07-28 Thread aaron at aaronballman dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110848 --- Comment #3 from Aaron Ballman --- (In reply to Andrew Pinski from comment #1) > Since VLA support has been a GNU C++ extension way before it was proposed to > WG21, I doubt we want to enable this by default. I think it boils down to

[Bug c++/110848] Consider enabling -Wvla by default in C++ modes

2023-07-28 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110848 --- Comment #2 from Andrew Pinski --- GCC has documented VLA extensions for C++ support since r0-35216-g4b404517536c85 (PR 930 which was done in 2001). So

[Bug c++/110848] Consider enabling -Wvla by default in C++ modes

2023-07-28 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110848 Andrew Pinski changed: What|Removed |Added Severity|normal |enhancement Keywords|