[cfe-commits] [PATCH] Implement -Wvla correctly

Dmitri Gribenko gribozavr at gmail.com
Mon Jan 7 15:00:50 PST 2013


Hello,

GCC implements -Wvla as "warn on every VLA" (this is useful to find
every VLA, for example, if they are forbidden by coding guidelines).
Currently Clang implements -Wvla as "warn on VLA when it is an
extension".

The attached patch makes our behavior match GCC.  The existing vla
extwarn is moved under -Wvla-extension and is still included into
-Wgnu.

-Wvla is disabled by default, so I don't think that this change will
cause much fallout.

This fixes PR5953.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: implement-wvla-correctly-v1.patch
Type: application/octet-stream
Size: 3397 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130108/e88844ce/attachment.obj>


More information about the cfe-commits mailing list