On Wed, Jan 4, 2023 at 11:54 AM Nathan Chancellor <nathan@xxxxxxxxxx> wrote: > > A future change will switch as-option to use KBUILD_AFLAGS instead of > KBUILD_CFLAGS to allow clang to drop -Qunused-arguments, which may cause > issues if the flag being tested requires a flag previously added to > KBUILD_CFLAGS but not KBUILD_AFLAGS. Use cc-option for cflags additions > so that the flags are tested properly. > > Signed-off-by: Nathan Chancellor <nathan@xxxxxxxxxx> We don't need the trailing comma in the `call`, but it's not worth a v2 IMO. Thanks for the patch. Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> > --- > Cc: tsbogend@xxxxxxxxxxxxxxxx > Cc: linux-mips@xxxxxxxxxxxxxxx > --- > arch/mips/Makefile | 2 +- > arch/mips/loongson2ef/Platform | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/mips/Makefile b/arch/mips/Makefile > index a00a6d94e16f..04e46ec24319 100644 > --- a/arch/mips/Makefile > +++ b/arch/mips/Makefile > @@ -143,7 +143,7 @@ cflags-y += -fno-stack-check > # > # Avoid this by explicitly disabling that assembler behaviour. > # > -cflags-y += $(call as-option,-Wa$(comma)-mno-fix-loongson3-llsc,) > +cflags-y += $(call cc-option,-Wa$(comma)-mno-fix-loongson3-llsc,) > > # > # CPU-dependent compiler/assembler options for optimization. > diff --git a/arch/mips/loongson2ef/Platform b/arch/mips/loongson2ef/Platform > index eebabf9df6ac..c6f7a4b95997 100644 > --- a/arch/mips/loongson2ef/Platform > +++ b/arch/mips/loongson2ef/Platform > @@ -25,7 +25,7 @@ cflags-$(CONFIG_CPU_LOONGSON2F) += -march=loongson2f > # binutils does not merge support for the flag then we can revisit & remove > # this later - for now it ensures vendor toolchains don't cause problems. > # > -cflags-$(CONFIG_CPU_LOONGSON2EF) += $(call as-option,-Wa$(comma)-mno-fix-loongson3-llsc,) > +cflags-$(CONFIG_CPU_LOONGSON2EF) += $(call cc-option,-Wa$(comma)-mno-fix-loongson3-llsc,) > > # Enable the workarounds for Loongson2f > ifdef CONFIG_CPU_LOONGSON2F_WORKAROUNDS > > -- > 2.39.0 -- Thanks, ~Nick Desaulniers
