[PATCH] powerpc: Fix fatal warnings flag for LLVM's integrated assembler
When building with LLVM_IAS=1, there is an error because '-fatal-warnings' is not recognized as a valid flag: clang: error: unsupported argument '-fatal-warnings' to option '-Wa,' Use the double hyphen version of the flag, '--fatal-warnings', which works with both the GNU assembler and LLVM's integrated assembler. Fixes: 608d4a5ca563 ("powerpc: Error on assembly warnings") Signed-off-by: Nathan Chancellor --- arch/powerpc/Kbuild | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/Kbuild b/arch/powerpc/Kbuild index da862e9558bc..571f260b0842 100644 --- a/arch/powerpc/Kbuild +++ b/arch/powerpc/Kbuild @@ -1,6 +1,6 @@ # SPDX-License-Identifier: GPL-2.0 -subdir-ccflags-$(CONFIG_PPC_WERROR) := -Werror -Wa,-fatal-warnings -subdir-asflags-$(CONFIG_PPC_WERROR) := -Wa,-fatal-warnings +subdir-ccflags-$(CONFIG_PPC_WERROR) := -Werror -Wa,--fatal-warnings +subdir-asflags-$(CONFIG_PPC_WERROR) := -Wa,--fatal-warnings obj-y += kernel/ obj-y += mm/ --- base-commit: bfe51886ca544956eb4ff924d1937ac01d0ca9c8 change-id: 20240405-ppc-fix-wa-fatal-warnings-clang-603f0ebb0133 Best regards, -- Nathan Chancellor
Re: [PATCH] powerpc: Fix fatal warnings flag for LLVM's integrated assembler
On Fri, 05 Apr 2024 12:31:22 -0700, Nathan Chancellor wrote: > When building with LLVM_IAS=1, there is an error because > '-fatal-warnings' is not recognized as a valid flag: > > clang: error: unsupported argument '-fatal-warnings' to option '-Wa,' > > Use the double hyphen version of the flag, '--fatal-warnings', which > works with both the GNU assembler and LLVM's integrated assembler. > > [...] Applied to powerpc/next. [1/1] powerpc: Fix fatal warnings flag for LLVM's integrated assembler https://git.kernel.org/powerpc/c/8884fc918f6aee220f9b41806974508bd0213aca cheers
Re: [PATCH] powerpc: Fix fatal warnings flag for LLVM's integrated assembler
On Fri, Apr 5, 2024 at 12:31 PM Nathan Chancellor wrote: > > When building with LLVM_IAS=1, there is an error because > '-fatal-warnings' is not recognized as a valid flag: > > clang: error: unsupported argument '-fatal-warnings' to option '-Wa,' > > Use the double hyphen version of the flag, '--fatal-warnings', which > works with both the GNU assembler and LLVM's integrated assembler. > > Fixes: 608d4a5ca563 ("powerpc: Error on assembly warnings") > Signed-off-by: Nathan Chancellor Nice catch. Reviewed-by: Justin Stitt > --- > arch/powerpc/Kbuild | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/Kbuild b/arch/powerpc/Kbuild > index da862e9558bc..571f260b0842 100644 > --- a/arch/powerpc/Kbuild > +++ b/arch/powerpc/Kbuild > @@ -1,6 +1,6 @@ > # SPDX-License-Identifier: GPL-2.0 > -subdir-ccflags-$(CONFIG_PPC_WERROR) := -Werror -Wa,-fatal-warnings > -subdir-asflags-$(CONFIG_PPC_WERROR) := -Wa,-fatal-warnings > +subdir-ccflags-$(CONFIG_PPC_WERROR) := -Werror -Wa,--fatal-warnings > +subdir-asflags-$(CONFIG_PPC_WERROR) := -Wa,--fatal-warnings > > obj-y += kernel/ > obj-y += mm/ > > --- > base-commit: bfe51886ca544956eb4ff924d1937ac01d0ca9c8 > change-id: 20240405-ppc-fix-wa-fatal-warnings-clang-603f0ebb0133 > > Best regards, > -- > Nathan Chancellor >
Re: [PATCH] powerpc: Fix fatal warnings flag for LLVM's integrated assembler
Nathan Chancellor writes: > When building with LLVM_IAS=1, there is an error because > '-fatal-warnings' is not recognized as a valid flag: > > clang: error: unsupported argument '-fatal-warnings' to option '-Wa,' > > Use the double hyphen version of the flag, '--fatal-warnings', which > works with both the GNU assembler and LLVM's integrated assembler. > > Fixes: 608d4a5ca563 ("powerpc: Error on assembly warnings") Thanks. I guess I need to do some of my clang builds with LLVM_IAS=1 :} cheers