Martin Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/81659?usp=email )
Change subject: Kconfig: Reverse ARCH_SUPPORTS_CLANG ......................................................................
Kconfig: Reverse ARCH_SUPPORTS_CLANG
Since most targets support clang it's easier to reverse the semantics of the Kconfig options.
Change-Id: Ib28e7a4cb286b9f8b05be94dae3947179f43c746 Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/81659 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Nico Huber nico.h@gmx.de Reviewed-by: Maximilian Brune maximilian.brune@9elements.com --- M src/Kconfig M src/arch/arm/Kconfig M src/arch/arm64/Kconfig M src/arch/ppc64/Kconfig M src/arch/riscv/Kconfig M src/arch/x86/Kconfig 6 files changed, 7 insertions(+), 8 deletions(-)
Approvals: build bot (Jenkins): Verified Maximilian Brune: Looks good to me, approved Nico Huber: Looks good to me, but someone else must approve
diff --git a/src/Kconfig b/src/Kconfig index ef3c72c..b6dc67f 100644 --- a/src/Kconfig +++ b/src/Kconfig @@ -74,7 +74,7 @@
config COMPILER_LLVM_CLANG bool "LLVM/clang" - depends on ALLOW_EXPERIMENTAL_CLANG || ARCH_SUPPORTS_CLANG + depends on ALLOW_EXPERIMENTAL_CLANG || !CLANG_UNSUPPORTED help Use LLVM/clang to build coreboot. To use this, you must build the coreboot version of the clang compiler. Run the command @@ -85,15 +85,15 @@
endchoice
-config ARCH_SUPPORTS_CLANG +config CLANG_UNSUPPORTED bool help - Opt-in flag for architectures that generally work well with CLANG. - By default the option would be hidden. + Set this flag on platforms that do not support building with the + clang compiler.
config ALLOW_EXPERIMENTAL_CLANG bool "Allow experimental LLVM/Clang" - depends on !ARCH_SUPPORTS_CLANG + depends on CLANG_UNSUPPORTED help On some architectures CLANG does not work that well. Use this only to try to get CLANG working. diff --git a/src/arch/arm/Kconfig b/src/arch/arm/Kconfig index 64fe915..0829dcb 100644 --- a/src/arch/arm/Kconfig +++ b/src/arch/arm/Kconfig @@ -2,6 +2,7 @@
config ARCH_ARM bool + select CLANG_UNSUPPORTED
config ARCH_BOOTBLOCK_ARM bool diff --git a/src/arch/arm64/Kconfig b/src/arch/arm64/Kconfig index af5050b..ea7b5aa 100644 --- a/src/arch/arm64/Kconfig +++ b/src/arch/arm64/Kconfig @@ -2,6 +2,7 @@
config ARCH_ARM64 bool + select CLANG_UNSUPPORTED
config ARCH_BOOTBLOCK_ARM64 bool diff --git a/src/arch/ppc64/Kconfig b/src/arch/ppc64/Kconfig index 93e4929..25a0f50 100644 --- a/src/arch/ppc64/Kconfig +++ b/src/arch/ppc64/Kconfig @@ -2,7 +2,6 @@
config ARCH_PPC64 bool - select ARCH_SUPPORTS_CLANG
config ARCH_BOOTBLOCK_PPC64 bool diff --git a/src/arch/riscv/Kconfig b/src/arch/riscv/Kconfig index b570b01..66c64c2 100644 --- a/src/arch/riscv/Kconfig +++ b/src/arch/riscv/Kconfig @@ -10,7 +10,6 @@
config ARCH_RISCV bool - select ARCH_SUPPORTS_CLANG
if ARCH_RISCV
diff --git a/src/arch/x86/Kconfig b/src/arch/x86/Kconfig index c0fe6dc..16d8a70 100644 --- a/src/arch/x86/Kconfig +++ b/src/arch/x86/Kconfig @@ -5,7 +5,6 @@ select PCI select RELOCATABLE_MODULES select HAVE_ASAN_IN_RAMSTAGE - select ARCH_SUPPORTS_CLANG
if ARCH_X86