Skip to content

Revert "cross compile riscv64 with LLVM toolchain and sysroot (#1116)"#1150

Merged
zanieb merged 1 commit into
mainfrom
riscv_gcc
Jun 11, 2026
Merged

Revert "cross compile riscv64 with LLVM toolchain and sysroot (#1116)"#1150
zanieb merged 1 commit into
mainfrom
riscv_gcc

Conversation

@jjhelmus

Copy link
Copy Markdown
Contributor

This reverts commit 96a589a.

Having a split LLVM / older GNU ld toolchain causes issues with relocation symbols and truncations.

@jjhelmus jjhelmus added platform:linux Specific to the Linux platform arch:riscv64 python:all Select all Python versions build:all labels Jun 10, 2026
@jjhelmus jjhelmus closed this Jun 11, 2026
@jjhelmus jjhelmus reopened this Jun 11, 2026
@zanie-oai

Copy link
Copy Markdown

Why are you running things like this with all Python versions and build variants? jw, seems unnecessary and want to understand if the subsetting is working

@zanieb zanieb merged commit f1d7b92 into main Jun 11, 2026
73 of 79 checks passed
@zanieb zanieb deleted the riscv_gcc branch June 11, 2026 03:49
@zanieb

zanieb commented Jun 11, 2026

Copy link
Copy Markdown
Member

Merging assuming this is blocking release

@jjhelmus

Copy link
Copy Markdown
Contributor Author

Why are you running things like this with all Python versions and build variants?

main was failing on a single build of riscv64 due to relocation relaxation. The specifics of this were highly dependent on minor changes, 3.13 was failing before the upgrade to 3.13.14. 3.14 failed with the update to 3.14.6, the DER parser modification caused more builds to fail.

I wanted to ensure that reverting to GCC did not introduce new failures and fixed and fixed the current ones. The best method seemed to be to test everything that could be effected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arch:riscv64 build:all platform:linux Specific to the Linux platform python:all Select all Python versions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants