On Fri, Jan 17, 2020 at 8:33 PM Catalin Marinas <catalin.marinas@xxxxxxx> wrote: > > On Mon, Dec 16, 2019 at 02:17:16PM +0530, Amit Daniel Kachhap wrote: > > diff --git a/scripts/Kconfig.include b/scripts/Kconfig.include > > index d4adfbe..cc465dd 100644 > > --- a/scripts/Kconfig.include > > +++ b/scripts/Kconfig.include > > @@ -31,6 +31,10 @@ cc-option = $(success,$(CC) -Werror $(CLANG_FLAGS) $(1) -E -x c /dev/null -o /de > > # Return y if the linker supports <flag>, n otherwise > > ld-option = $(success,$(LD) -v $(1)) > > > > +# $(as-option,<flag>) > > +# Return y if the assembler supports <flag>, n otherwise > > +as-option = $(success, $(CC) $(CLANG_FLAGS) $(1) -E -x assembler /dev/null -o /dev/null) > > I had different experiments with this for MTE and noticed that clang > does not honour the -Wa, option (which you use in a subsequent patch). > So not sure how useful as-option is. I think this is because it uses '-E' option. To invoke the assembler, -c is needed. I replaced -E with -c, and tested it. It seems working for both gcc and clang. I noticed a similar case for cc-option: https://patchwork.kernel.org/patch/11339567/ -- Best Regards Masahiro Yamada
