llext: arc: enforce bitness match

laurenmurphyx64 · kartben · commit f0eb7ebc8305 · 2025-02-05T21:01:01.000+01:00
Enforces bitness match of ARC compilers and linkers invoked building LLEXT apps by inheriting -mcpu flag from Zephyr. The ARC toolchain becomes confused without the -mcpu flag. On qemu_hs5x we see GCC select elf32-littlearc64 for linking an object previously compiled as elf64-littlearc64. Fixes #80949 Signed-off-by: Lauren Murphy <lauren.murphy@intel.com>
diff --git a/cmake/compiler/gcc/target_arc.cmake b/cmake/compiler/gcc/target_arc.cmake
@@ -18,5 +18,9 @@ set(LLEXT_REMOVE_FLAGS
   -Os
 )
 
+set(LLEXT_APPEND_FLAGS
+  -mcpu=${GCC_ARC_TUNED_CPU} # Force compiler and linker match
+)
+
 list(APPEND TOOLCHAIN_C_FLAGS -mcpu=${GCC_ARC_TUNED_CPU})
 list(APPEND TOOLCHAIN_LD_FLAGS -mcpu=${GCC_ARC_TUNED_CPU})
diff --git a/tests/subsys/llext/CMakeLists.txt b/tests/subsys/llext/CMakeLists.txt
@@ -75,6 +75,7 @@ if (CONFIG_LLEXT_TYPE_ELF_RELOCATABLE AND NOT CONFIG_ARM AND NOT CONFIG_RISCV)
     TARGET pre_located_ext
     POST_BUILD
     COMMAND ${CMAKE_C_COMPILER}
+    ${LLEXT_APPEND_FLAGS}
     -Wl,-r -Wl,-Ttext=0xbada110c -nostdlib -nodefaultlibs -nostartfiles
     $<TARGET_OBJECTS:${pre_located_target}> -o ${pre_located_file}
   )

Original file line number	Diff line number	Diff line change
`@@ -18,5 +18,9 @@ set(LLEXT_REMOVE_FLAGS`
`18`	`18`	`-Os`
`19`	`19`	`)`
`20`	`20`
	`21`	`+set(LLEXT_APPEND_FLAGS`
	`22`	`+ -mcpu=${GCC_ARC_TUNED_CPU} # Force compiler and linker match`
	`23`	`+)`
	`24`	`+`
`21`	`25`	`list(APPEND TOOLCHAIN_C_FLAGS -mcpu=${GCC_ARC_TUNED_CPU})`
`22`	`26`	`list(APPEND TOOLCHAIN_LD_FLAGS -mcpu=${GCC_ARC_TUNED_CPU})`
Original file line number	Diff line number	Diff line change
`@@ -75,6 +75,7 @@ if (CONFIG_LLEXT_TYPE_ELF_RELOCATABLE AND NOT CONFIG_ARM AND NOT CONFIG_RISCV)`
`75`	`75`	`TARGET pre_located_ext`
`76`	`76`	`POST_BUILD`
`77`	`77`	`COMMAND ${CMAKE_C_COMPILER}`
	`78`	`+ ${LLEXT_APPEND_FLAGS}`
`78`	`79`	`-Wl,-r -Wl,-Ttext=0xbada110c -nostdlib -nodefaultlibs -nostartfiles`
`79`	`80`	`$<TARGET_OBJECTS:${pre_located_target}> -o ${pre_located_file}`
`80`	`81`	`)`