-
-
Notifications
You must be signed in to change notification settings - Fork 313
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
Build roc with llvm musl #6083
Comments
Current state (copied from zulip): ================================ these are the instructions for reproduction and the error: git clone https://github.com/Anton-4/zig-bootstrap.git cd into roc-repo! use correct path for your caseexport LLVM_SYS_160_PREFIX=/home/username/gitrepos/zig-bootstrap/out/x86_64-linux-musl-baseline/ error: ❯ cargo build --release --bin roc [...]
= note: some error: could not compile The important part seems to be undefined reference to std::__1::basic_string...` I wonder if we also need to build and link against llvm libc++? Cause this seems like it could be a libc++ versioning issue, which could be related to trying to build static which means it can't dynamically pull in the system libc++. That said, a bit surprised it fails on roc and not during llvm compilation. That suggests that whatever it needs is defined somewhere in the llvm build process and we just aren't pulling it in (maybe not that surprising cause llvm-sys probably expects a dynamic llvm which automagically pulls in it's own dependencies). |
I think we determined this requires llvm 18 to work around a bug, do you remember who last did some work on llvm musl @lukewilliamboswell? Do we have a branch for it? |
We should be good to use musl once we upgrade to llvm 18. Ryan and I were working on that, and I think the issue was a bug or something in 16 which prevented us from using it. |
The llvm upgrade branch is mostly done, and least the bulk of the upgrade changes for llvm and zig etc. The latest was it needed some CI machine stuff to land, and there was a couple of smaller things. The plan was to merge the changes after the prebuilt host PR. |
We've talked about this a lot but it's nice to have an issue for tracking.
The text was updated successfully, but these errors were encountered: