Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-04-11 Thread Masahiro Yamada
2017-04-01 5:38 GMT+09:00 Arnd Bergmann : > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both compiler behave

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-04-11 Thread Masahiro Yamada
2017-04-01 5:38 GMT+09:00 Arnd Bergmann : > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both compiler behave the same. > > [arnd: it turns out

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-04-05 Thread Masahiro Yamada
Hi Arnd, 2017-04-03 6:46 GMT+09:00 Masahiro Yamada : > Hi Arnd, > > > 2017-04-01 5:38 GMT+09:00 Arnd Bergmann : >> From: Mark Charlebois >> >> Clang will warn about unknown warnings but will not return false >> unless -Werror is

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-04-05 Thread Masahiro Yamada
Hi Arnd, 2017-04-03 6:46 GMT+09:00 Masahiro Yamada : > Hi Arnd, > > > 2017-04-01 5:38 GMT+09:00 Arnd Bergmann : >> From: Mark Charlebois >> >> Clang will warn about unknown warnings but will not return false >> unless -Werror is set. GCC will return false if an unknown >> warning is passed. >>

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-04-02 Thread Masahiro Yamada
Hi Arnd, 2017-04-01 5:38 GMT+09:00 Arnd Bergmann : > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-04-02 Thread Masahiro Yamada
Hi Arnd, 2017-04-01 5:38 GMT+09:00 Arnd Bergmann : > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both compiler behave the same. > > [arnd: it

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-03-31 Thread Kees Cook
On Fri, Mar 31, 2017 at 1:38 PM, Arnd Bergmann wrote: > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-03-31 Thread Kees Cook
On Fri, Mar 31, 2017 at 1:38 PM, Arnd Bergmann wrote: > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both compiler behave the same. > > [arnd:

[PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-03-31 Thread Arnd Bergmann
From: Mark Charlebois Clang will warn about unknown warnings but will not return false unless -Werror is set. GCC will return false if an unknown warning is passed. Adding -Werror make both compiler behave the same. [arnd: it turns out we need the same patch for testing

[PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2017-03-31 Thread Arnd Bergmann
From: Mark Charlebois Clang will warn about unknown warnings but will not return false unless -Werror is set. GCC will return false if an unknown warning is passed. Adding -Werror make both compiler behave the same. [arnd: it turns out we need the same patch for testing whether

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-25 Thread Behan Webster
On 09/25/14 06:34, Michal Marek wrote: On 2014-09-24 20:50, Behan Webster wrote: Getting clang to error on unused flags wasn't trivial (this change broke a lot of builds apparently). Fortunately we weren't the only ones who wanted it to behave like gcc in this case. I think it's going to be

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-25 Thread Michal Marek
On 2014-09-24 20:50, Behan Webster wrote: > On 09/24/14 05:07, Michal Marek wrote: >> On 2014-09-23 21:28, beh...@converseincode.com wrote: >>> From: Mark Charlebois >>> >>> Clang will warn about unknown warnings but will not return false >> You mean unknown options, right? > 2 kinds of options:

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-25 Thread Behan Webster
On 09/25/14 06:34, Michal Marek wrote: On 2014-09-24 20:50, Behan Webster wrote: Getting clang to error on unused flags wasn't trivial (this change broke a lot of builds apparently). Fortunately we weren't the only ones who wanted it to behave like gcc in this case. I think it's going to be

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-25 Thread Michal Marek
On 2014-09-24 20:50, Behan Webster wrote: On 09/24/14 05:07, Michal Marek wrote: On 2014-09-23 21:28, beh...@converseincode.com wrote: From: Mark Charlebois charl...@gmail.com Clang will warn about unknown warnings but will not return false You mean unknown options, right? 2 kinds of

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-24 Thread Behan Webster
On 09/24/14 05:07, Michal Marek wrote: On 2014-09-23 21:28, beh...@converseincode.com wrote: From: Mark Charlebois Clang will warn about unknown warnings but will not return false You mean unknown options, right? 2 kinds of options: flags and warnings. clang used to merely warn about

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-24 Thread Michal Marek
On 2014-09-23 21:28, beh...@converseincode.com wrote: > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false You mean unknown options, right? > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-24 Thread Michal Marek
On 2014-09-23 21:28, beh...@converseincode.com wrote: From: Mark Charlebois charl...@gmail.com Clang will warn about unknown warnings but will not return false You mean unknown options, right? unless -Werror is set. GCC will return false if an unknown warning is passed. Adding -Werror

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-24 Thread Behan Webster
On 09/24/14 05:07, Michal Marek wrote: On 2014-09-23 21:28, beh...@converseincode.com wrote: From: Mark Charlebois charl...@gmail.com Clang will warn about unknown warnings but will not return false You mean unknown options, right? 2 kinds of options: flags and warnings. clang used to merely

[PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-23 Thread behanw
From: Mark Charlebois Clang will warn about unknown warnings but will not return false unless -Werror is set. GCC will return false if an unknown warning is passed. Adding -Werror make both compiler behave the same. Signed-off-by: Mark Charlebois Signed-off-by: Behan Webster Reviewed-by:

[PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-23 Thread behanw
From: Mark Charlebois charl...@gmail.com Clang will warn about unknown warnings but will not return false unless -Werror is set. GCC will return false if an unknown warning is passed. Adding -Werror make both compiler behave the same. Signed-off-by: Mark Charlebois charl...@gmail.com