Skip to content

Update Clang toolchain used by Bazel#2879

Merged
kilograham merged 2 commits into
raspberrypi:developfrom
Prabhuk:update_toolchain_mar20
Jun 29, 2026
Merged

Update Clang toolchain used by Bazel#2879
kilograham merged 2 commits into
raspberrypi:developfrom
Prabhuk:update_toolchain_mar20

Conversation

@Prabhuk

@Prabhuk Prabhuk commented Mar 25, 2026

Copy link
Copy Markdown

The Clang toolchain is updated to a newer revision 9431920bfaee50f4b1552fc01ea081a5c18a5a3b from March 20, 2026.

Fixes #2859

The Clang toolchain is updated to a newer revision 9431920bfaee50f4b1552fc01ea081a5c18a5a3b from March 20, 2026.

Fixes raspberrypi#2859
@github-actions

Copy link
Copy Markdown

Please do not submit against master, use develop instead

@Prabhuk Prabhuk changed the base branch from master to develop March 31, 2026 16:36
@Prabhuk

Prabhuk commented Mar 31, 2026

Copy link
Copy Markdown
Author

CC: @petrhosek @kilograham

@kilograham kilograham added this to the 2.2.1 milestone Apr 1, 2026
Comment thread cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake Outdated
@@ -1,3 +1,5 @@
set(PICO_HARD_FLOAT_ABI 1 CACHE BOOL "Use hard floating point ABI")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This changes the default PICO_HARD_FLOAT_ABI for all Clang users, which I don't think is necessary for this PR - all the other Clang toolchains still support both hard and soft float ABIs, it's just the new Google toolchain that only supports hard float.

A better fix might be to remove armv8m.main-unknown-none-eabi from the softfp PICO_CLANG_RUNTIMES list, which should then throw the Could not find an llvm runtime error if PICO_HARD_FLOAT_ABI=0 (assuming support for the older soft float toolchain is not required).

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. PTAL

Comment thread cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake Outdated
Comment thread cmake/preload/toolchains/pico_arm_cortex_m33_clang.cmake Outdated
@Prabhuk Prabhuk force-pushed the update_toolchain_mar20 branch from 81b8de1 to 74e55a4 Compare June 29, 2026 16:55
@Prabhuk

Prabhuk commented Jun 29, 2026

Copy link
Copy Markdown
Author

As discussed offline, we are bumping the toolchain revision to a new enough revision but prior to toolchain switching the float ABI for cortex-m33 runtime libraries to hard float.

@kilograham kilograham self-requested a review June 29, 2026 17:03
@kilograham kilograham merged commit baec39b into raspberrypi:develop Jun 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update clang toolchain revision in Bazel builds

4 participants