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

Bump LLVM version to 18 #13

Closed
wants to merge 1 commit into from
Closed

Bump LLVM version to 18 #13

wants to merge 1 commit into from

Conversation

AdvenamTacet
Copy link
Contributor

@AdvenamTacet AdvenamTacet commented Feb 26, 2024

This commit changes clang installed directly from apt to clang installed by an install LLVM apt script.
It installs LLVM18 (not reproducible).

To make it work easily, it was necessary to change Debian to Ubuntu.


Reasoning behind using LLVM18:

Currently, ASAN will only work correctly when using a recent head build of LLVM/Clang - it requires this bugfix related to multithreaded fork, which is not yet in any released version. See here for instructions on how to build LLVM/Clang from source (note you will need at least the clang and compiler-rt projects enabled). Then, you will need to replace CC=clang in the instructions with an explicit path to your built Clang binary.

https://github.com/ruby/ruby/blob/master/doc/contributing/building_ruby.md#building-with-address-sanitizer

This commit changes clang installed directly from apt to clang installed
by an install LLVM apt script.
It installs LLVM18 (not reproducible).

To make it work easily, it was necessary to change Debian to Ubuntu.
@mschwager
Copy link
Member

I think #16 should support this use-case, and also choosing different versions using the LLVM_VERSION build arg

@mschwager mschwager closed this Mar 28, 2024
@mschwager mschwager deleted the update-llvm-to-18 branch March 28, 2024 11:53
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