Re: [PATCH] kbuild: add -Wvla flag unconditionally
On Fri, May 10, 2019 at 1:52 AM Nick Desaulniers wrote: > > > On Wed, May 8, 2019 at 11:46 PM Masahiro Yamada > > wrote: > > > This flag is documented in the GCC 4.6 manual, and recognized by > > > Clang as well. Let's rip off the cc-option switch. > > Checked w/ godbolt w/ Clang 4 and GCC 4.6.4. > Tested-by: Nick Desaulniers > > -- > Thanks, > ~Nick Desaulniers Applied to linux-kbuild. -- Best Regards Masahiro Yamada
Re: [PATCH] kbuild: add -Wvla flag unconditionally
> On Wed, May 8, 2019 at 11:46 PM Masahiro Yamada > wrote: > > This flag is documented in the GCC 4.6 manual, and recognized by > > Clang as well. Let's rip off the cc-option switch. Checked w/ godbolt w/ Clang 4 and GCC 4.6.4. Tested-by: Nick Desaulniers -- Thanks, ~Nick Desaulniers
Re: [PATCH] kbuild: add -Wvla flag unconditionally
On Wed, May 8, 2019 at 11:46 PM Masahiro Yamada wrote: > This flag is documented in the GCC 4.6 manual, and recognized by > Clang as well. Let's rip off the cc-option switch. > > Signed-off-by: Masahiro Yamada Acked-by: Kees Cook -- Kees Cook
Re: [PATCH] kbuild: add -Wvla flag unconditionally
On Thu, May 09, 2019 at 03:45:49PM +0900, Masahiro Yamada wrote: > This flag is documented in the GCC 4.6 manual, and recognized by > Clang as well. Let's rip off the cc-option switch. > > Signed-off-by: Masahiro Yamada Reviewed-by: Nathan Chancellor > --- > > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index c71ffb6f55b5..3f5e8a0f4561 100644 > --- a/Makefile > +++ b/Makefile > @@ -841,7 +841,7 @@ NOSTDINC_FLAGS += -nostdinc -isystem $(shell $(CC) > -print-file-name=include) > KBUILD_CFLAGS += -Wdeclaration-after-statement > > # Variable Length Arrays (VLAs) should not be used anywhere in the kernel > -KBUILD_CFLAGS += $(call cc-option,-Wvla) > +KBUILD_CFLAGS += -Wvla > > # disable pointer signed / unsigned warnings in gcc 4.0 > KBUILD_CFLAGS += -Wno-pointer-sign > -- > 2.17.1 >
Re: [PATCH] kbuild: add -Wvla flag unconditionally
On Thu, May 9, 2019 at 8:46 AM Masahiro Yamada wrote: > > This flag is documented in the GCC 4.6 manual, and recognized by > Clang as well. Let's rip off the cc-option switch. > [ CC Kees who did the VLA removal/cleanup ] Looks good to me. Reviewed-by: Sedat Dilek > Signed-off-by: Masahiro Yamada > --- > > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index c71ffb6f55b5..3f5e8a0f4561 100644 > --- a/Makefile > +++ b/Makefile > @@ -841,7 +841,7 @@ NOSTDINC_FLAGS += -nostdinc -isystem $(shell $(CC) > -print-file-name=include) > KBUILD_CFLAGS += -Wdeclaration-after-statement > > # Variable Length Arrays (VLAs) should not be used anywhere in the kernel > -KBUILD_CFLAGS += $(call cc-option,-Wvla) > +KBUILD_CFLAGS += -Wvla > > # disable pointer signed / unsigned warnings in gcc 4.0 > KBUILD_CFLAGS += -Wno-pointer-sign > -- > 2.17.1 > > -- > You received this message because you are subscribed to the Google Groups > "Clang Built Linux" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to clang-built-linux+unsubscr...@googlegroups.com. > To post to this group, send email to clang-built-li...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/clang-built-linux/20190509064549.1302-1-yamada.masahiro%40socionext.com. > For more options, visit https://groups.google.com/d/optout.
[PATCH] kbuild: add -Wvla flag unconditionally
This flag is documented in the GCC 4.6 manual, and recognized by Clang as well. Let's rip off the cc-option switch. Signed-off-by: Masahiro Yamada --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index c71ffb6f55b5..3f5e8a0f4561 100644 --- a/Makefile +++ b/Makefile @@ -841,7 +841,7 @@ NOSTDINC_FLAGS += -nostdinc -isystem $(shell $(CC) -print-file-name=include) KBUILD_CFLAGS += -Wdeclaration-after-statement # Variable Length Arrays (VLAs) should not be used anywhere in the kernel -KBUILD_CFLAGS += $(call cc-option,-Wvla) +KBUILD_CFLAGS += -Wvla # disable pointer signed / unsigned warnings in gcc 4.0 KBUILD_CFLAGS += -Wno-pointer-sign -- 2.17.1