From 4df926b02de0e64d976103dcb9829b632fc1f4f3 Mon Sep 17 00:00:00 2001 From: Nicolas Lebedenco Date: Thu, 28 Mar 2024 23:37:01 -0400 Subject: [PATCH] cmake: Make enable_language() clear about what it is actually doing That call to enable_languages() is passed C CXX ASM which misleads the reader into thinking that the three languages are enabled in that call but at that point only ASM is actually because C and CXX were already implicitly enabled by the call to project() above. The suggested change removes the misleading C CXX arguments but we could as well make it clear in the project() call by passing LANGUAGES C CXX. Signed-off-by: Nicolas Lebedenco --- cmake/modules/kernel.cmake | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmake/modules/kernel.cmake b/cmake/modules/kernel.cmake index a093d46691f..35dda13b26b 100644 --- a/cmake/modules/kernel.cmake +++ b/cmake/modules/kernel.cmake @@ -131,7 +131,7 @@ project(Zephyr-Kernel VERSION ${PROJECT_VERSION}) # because clang from OneApi can't recognize them as asm files on # windows now. list(APPEND CMAKE_ASM_SOURCE_FILE_EXTENSIONS "S") -enable_language(C CXX ASM) +enable_language(ASM) # The setup / configuration of the toolchain itself and the configuration of # supported compilation flags are now split, as this allows to use the toolchain