From 48b366f39d7681f0b4b81f24344d16aba5368a0c Mon Sep 17 00:00:00 2001 From: graham sanderson Date: Mon, 3 Feb 2025 16:06:38 -0600 Subject: [PATCH 1/2] workaround change in llvm picked up by llvm ETA 19 --- src/rp2_common/pico_clib_interface/picolibc_interface.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/rp2_common/pico_clib_interface/picolibc_interface.c b/src/rp2_common/pico_clib_interface/picolibc_interface.c index 728e93a56..63d79135e 100644 --- a/src/rp2_common/pico_clib_interface/picolibc_interface.c +++ b/src/rp2_common/pico_clib_interface/picolibc_interface.c @@ -7,6 +7,8 @@ #include #include #include +#include +#include #include #include From b939e0e0477a9c84d372a744d4e8bd51816fe950 Mon Sep 17 00:00:00 2001 From: graham sanderson Date: Mon, 3 Feb 2025 16:47:55 -0600 Subject: [PATCH 2/2] fix clang compilation not including floating point instructions in build output --- cmake/preload/toolchains/pico_arm_clang_arm.cmake | 2 -- cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake | 2 +- src/cmake/on_device.cmake | 3 +-- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/cmake/preload/toolchains/pico_arm_clang_arm.cmake b/cmake/preload/toolchains/pico_arm_clang_arm.cmake index bf06d3410..4c7816561 100644 --- a/cmake/preload/toolchains/pico_arm_clang_arm.cmake +++ b/cmake/preload/toolchains/pico_arm_clang_arm.cmake @@ -1,5 +1,3 @@ -# NOTE: THIS IS A WIP ONLY PICO_ARM_GCC IS CURRENTLY SUPPORTED -# todo there is probably a more "cmake" way of doing this going thru the standard path with our "PICO" platform # i.e. CMakeInformation and whatnot include(${CMAKE_CURRENT_LIST_DIR}/util/find_compiler.cmake) diff --git a/cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake b/cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake index d35bdbc19..c5199e55a 100644 --- a/cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake +++ b/cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake @@ -4,5 +4,5 @@ set(CMAKE_SYSTEM_PROCESSOR cortex-m33) set(PICO_CLANG_RUNTIMES armv8m.main_soft_nofp armv8m.main-unknown-none-eabi) set(PICO_COMMON_LANG_FLAGS "-mcpu=cortex-m33 --target=armv8m.main-none-eabi -mfloat-abi=softfp -march=armv8m.main+fp+dsp") - +set(PICO_DISASM_OBJDUMP_ARGS --mcpu=cortex-m33 --arch=armv8m.main+fp+dsp) include(${CMAKE_CURRENT_LIST_DIR}/util/pico_arm_clang_common.cmake) diff --git a/src/cmake/on_device.cmake b/src/cmake/on_device.cmake index 5cabf3cc9..00a5fffc6 100644 --- a/src/cmake/on_device.cmake +++ b/src/cmake/on_device.cmake @@ -37,10 +37,9 @@ function(pico_add_dis_output TARGET) set(EXTRA_COMMAND COMMAND picotool coprodis --quiet ${output_path}$>,$,$>.dis ${output_path}$>,$,$>.dis) endif() endif() - add_custom_command(TARGET ${TARGET} POST_BUILD COMMAND ${CMAKE_OBJDUMP} -h $ > ${output_path}$>,$,$>.dis - COMMAND ${CMAKE_OBJDUMP} -d $ >> ${output_path}$>,$,$>.dis + COMMAND ${CMAKE_OBJDUMP} -d ${PICO_DISASM_OBJDUMP_ARGS} $ >> ${output_path}$>,$,$>.dis ${EXTRA_COMMAND} VERBATIM )