search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

-Wno-vla not supported #2732

Closed Quuxplusone closed 15 years ago

Quuxplusone commented 15 years ago
Bugzilla Link PR2508
Status RESOLVED INVALID
Importance P normal
Reported by Nick Lewycky (nlewycky@google.com)
Reported on 2008-06-30 17:31:46 -0700
Last modified on 2008-07-01 03:08:16 -0700
Version trunk
Hardware PC Linux
CC anton@korobeynikov.info, baldrick@free.fr, llvm-bugs@lists.llvm.org
Fixed by commit(s)
Attachments
Blocks
Blocked by
See also 
$ llvm-g++ -Wvla x.cpp
cc1plus: error: unrecognized command line option "-Wvla"

It'd be great if llvm-g++ would support the -Wvla (and -Wno-vla) option.
Quuxplusone commented 15 years ago 
(In reply to comment #0)
> $ llvm-g++ -Wvla x.cpp
> cc1plus: error: unrecognized command line option "-Wvla"
>
> It'd be great if llvm-g++ would support the -Wvla (and -Wno-vla) option.
This seems to be not supported even by mainline gcc:

$ gcc -v
Using built-in specs.
Target: i686-pc-linux-gnu
Configured with: /var/tmp/portage/sys-devel/gcc-4.2.2/work/gcc-4.2.2/configure -
-prefix=/usr --bindir=/usr/i686-pc-linux-gnu/gcc-bin/4.2.2 --
includedir=/usr/lib/gcc/i686-pc-linux-gnu/4.2.2/include --
datadir=/usr/share/gcc-data/i686-pc-linux-gnu/4.2.2 --mandir=/usr/share/gcc-
data/i686-pc-linux-gnu/4.2.2/man --infodir=/usr/share/gcc-data/i686-pc-linux-
gnu/4.2.2/info --with-gxx-include-dir=/usr/lib/gcc/i686-pc-linux-
gnu/4.2.2/include/g++-v4 --host=i686-pc-linux-gnu --build=i686-pc-linux-gnu --
disable-altivec --enable-nls --without-included-gettext --with-system-zlib --
disable-checking --disable-werror --enable-secureplt --disable-libunwind-
exceptions --disable-multilib --enable-libmudflap --disable-libssp --disable-
libgcj --with-arch=i686 --enable-languages=c,c++,fortran --enable-shared --
enable-threads=posix --enable-__cxa_atexit --enable-clocale=gnu
Thread model: posix
gcc version 4.2.2 (Gentoo 4.2.2 p1.0)

$ touch foo.c && gcc -Wvla foo.c
cc1: error: unrecognized command line option "-Wvla"
Quuxplusone commented 15 years ago

It's a GCC 4.3 feature, so it'd require an independent reimplementation, but I understand that such a reimplementation should be relatively simple for someone familiar with gcc.

Quuxplusone commented 15 years ago

Personally I'd rather work on porting llvm to gcc-4.4...