Thread (5 messages) 5 messages, 4 authors, 2018-12-16

Re: [PATCH] pinctrl: aspeed: Wrap -Woverride-init with cc-option

From: Nathan Chancellor <hidden>
Date: 2018-12-11 00:25:32
Also in: linux-aspeed, linux-gpio, lkml

On Tue, Dec 11, 2018 at 10:46:26AM +1030, Joel Stanley wrote:
On Tue, 11 Dec 2018 at 10:40, Andrew Jeffery [off-list ref] wrote:
quoted


On Tue, 11 Dec 2018, at 10:35, Nathan Chancellor wrote:
quoted
Clang does not support this option:

warning: unknown warning option '-Woverride-init'; did you mean
'-Woverride-module'? [-Wunknown-warning-option]
1 warning generated.

Signed-off-by: Nathan Chancellor <redacted>
Acked-by: Andrew Jeffery <redacted>
Thanks Nathan.

Andrew explained to me that this warning helps find bugs when
developing the complicated (some would say crazy) set of macros that
this pinctrl driver uses. It would be good to have this warning
available in clang. Do we have a clang issue opened for that?

Acked-by: Joel Stanley <joel@jms.id.au>

Cheers,

Joel
Hi Joel,

I just opened https://github.com/ClangBuiltLinux/linux/issues/293.

I searched the LLVM issue tracker and haven't see any reference of this.
I can file an issue but if you are a little more familiar with this
warning, you are welcome to do it instead.

Thanks for the quick response,
Nathan

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help