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

Use LLVM 16 #67

Merged
merged 1 commit into from
Apr 19, 2024
Merged

Use LLVM 16 #67

merged 1 commit into from
Apr 19, 2024

Conversation

radekdoulik
Copy link
Member

No description provided.

@radekdoulik
Copy link
Member Author

Let see if emscripten still builds with llvm 16

@akoeplinger
Copy link
Member

We can certainly try but there's a huge amount of libcxx changes in upstream that were also merged :)
I also wonder if that'd make third-party libraries incompatible if they were built with upstream emsdk instead of our version, even though they use the same version number.

@radekdoulik
Copy link
Member Author

@directhex what was the motivation to use older llvm versions?

@directhex
Copy link
Member

@directhex what was the motivation to use older llvm versions?

It's not an arbitrarily old version, it's (in theory!) the LLVM commit matching the binary emsdk release's bundle.

emsdk doesn't really have what you would call "release management" - if you build an old tag from source today, it uses git main HEAD for some components (like LLVM) so a from-source build today of 3.1.34 has entirely different libraries in it vs. the binary download of 3.1.34. It was also kinda synergy that LLVM 16 is also the version we use to build the product on Linux (https://github.com/dotnet/dotnet-buildtools-prereqs-docker/blob/main/src/cbl-mariner/2.0/crossdeps-builder/Dockerfile#L56) but I don't think it hugely matters that those align.

Copy link
Member

@lewing lewing left a comment

Choose a reason for hiding this comment

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

We can do this for the moment but we should also see about standing up a llvm19x build

@lewing lewing merged commit 4c94aa2 into dotnet:dotnet/main Apr 19, 2024
3 checks passed
radekdoulik added a commit to radekdoulik/emscripten that referenced this pull request May 23, 2024
radekdoulik added a commit that referenced this pull request May 23, 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.

4 participants