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

[cmake] Exposes LLVM version number in the runtimes. #84641

Merged

Conversation

mordante
Copy link
Member

@mordante mordante commented Mar 9, 2024

This allows sharing the LLVM version number in libc++.

This allows sharing the LLVM version number in libc++.
@mordante mordante requested review from tru and tstellar March 9, 2024 17:01
@mordante mordante requested a review from a team as a code owner March 9, 2024 17:01
@llvmbot llvmbot added the cmake Build system in general and CMake in particular label Mar 9, 2024
@mordante
Copy link
Member Author

mordante commented Mar 9, 2024

I originally used this in #81362, there it's not needed anymore. However @ldionne and I feel it would be useful to have this. We probably can use this to set the libc++ version number.

@mordante mordante merged commit 81e2047 into llvm:main Mar 11, 2024
50 of 52 checks passed
@mordante mordante deleted the review/expose_llvm_version_number_in_runtimes branch March 11, 2024 16:44
@firewave
Copy link

I wonder if this could help with #57263.

glandium added a commit to glandium/wasi-sdk that referenced this pull request Mar 12, 2024
abrown pushed a commit to WebAssembly/wasi-sdk that referenced this pull request Mar 12, 2024
@mordante
Copy link
Member Author

I wonder if this could help with #57263.

These CMake values were already available to clang-tidy. This made them available for the run-times.

fifield added a commit to Xilinx/mlir-aie that referenced this pull request Apr 3, 2024
fifield added a commit to Xilinx/mlir-aie that referenced this pull request Apr 4, 2024
fifield added a commit to Xilinx/mlir-aie that referenced this pull request Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmake Build system in general and CMake in particular
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants