Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Libffi] Temporary revert to 3.2.2 to build for aarch64-freebsd #10016

Merged
merged 2 commits into from
Dec 17, 2024

Conversation

JamesWrigley
Copy link
Contributor

@JamesWrigley JamesWrigley commented Dec 14, 2024

As discussed here: #10006 (comment)

This reverts the following commits:

Libffi has a more interesting commit history than Expat so I opted to revert all the commits newer than the change to 3.2.2 to be on the safe side. Interestingly, turns out that the JLL version 3.2.2 is actually building the library version 3.2.1.

EDIT: oops, #3906 wasn't a version change so that shouldn't have been reverted.

This reverts the following commits:
- "[Libffi] Rebuild for freebsd-aarch64 (JuliaPackaging#9909)":           670483c
- "Libffi: Build for aarch64-freebsd (JuliaPackaging#9782)":              f66e34e
- "Libffi: New version 3.4.6 (JuliaPackaging#8275)":                      ff6c554
- "[Libffi] Upgrade to v3.4.4 (JuliaPackaging#5834)":                     cca8463
@JamesWrigley
Copy link
Contributor Author

I'm guessing the compilers have been updated since last time this was built, on aarch64-apple-darwin the build was hitting this error: llvm/llvm-project#72802
I added a patch to fix it.

TL;DR: newer LLVM versions are pickier about the operations allowed in .cfi regions and it's not legal to declare labels within them anymore. Same thing happened to Julia: JuliaLang/julia#54634

@giordano giordano merged commit c82c599 into JuliaPackaging:master Dec 17, 2024
20 checks passed
@JamesWrigley JamesWrigley deleted the libffi-compat branch December 17, 2024 12:59
JamesWrigley added a commit to JamesWrigley/Yggdrasil that referenced this pull request Dec 17, 2024
giordano pushed a commit that referenced this pull request Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants