Skip to content

Add find compiler-rt libs cmake module for windows #52

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

Merged
merged 2 commits into from
Jan 14, 2024

Conversation

omjavaid
Copy link
Contributor

This patch adds find compiler-rt libs function imported from llvm-project/cmake/modules to testsuite repository. Some tests require to link compiler-rt builtins library on windows and this function helps locate the library path from install dir.

@omjavaid omjavaid requested a review from petrhosek November 16, 2023 10:35
This patch adds find compiler-rt libs function imported from
llvm-project/cmake/modules to testsuite repository. Some tests
require to link compiler-rt builtins library on windows and
this function helps locate the library path from install dir.
@omjavaid omjavaid requested a review from tarunprabhu December 20, 2023 16:01
Copy link
Contributor

@tarunprabhu tarunprabhu left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@omjavaid omjavaid merged commit 1c4054c into llvm:main Jan 14, 2024
@petrhosek
Copy link
Member

I missed this PR earlier, sorry about that. Why not directly copy https://github.com/llvm/llvm-project/blob/21edd381e403079baa2d01754e3d3ae7b3469932/cmake/Modules/HandleCompilerRT.cmake? That would have made it easier to keep the two modules in sync and I expect the support for finding compiler-rt on other platforms would be useful in test-suite as well.

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.

3 participants