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

[compiler-rt] Fix CMake configure on Windows #90843

Merged
merged 2 commits into from
May 2, 2024
Merged

Conversation

omjavaid
Copy link
Contributor

@omjavaid omjavaid commented May 2, 2024

CMake configure compiler-rt got broken as a result of following commit: d3925e6

This patch fixes the break by porting the above commit for clang-cl.

This problem was not caught on Windows buildbots beacuase it appeared when compiler-rt was included via LLVM_ENABLE_PROJECTS while buildbots include compiler-rt project using LLVM_ENABLE_RUNTIMES flag.

CMake configure compiler-rt got broken as a result of following commit:
d3925e6

This patch fixes the break by porting the above commit for clang-cl.

This problem was not caught on Windows buildbots beacuase it appeared
when compiler-rt was included via LLVM_ENABLE_PROJECTS while buildbots
include compiler-rt project using LLVM_ENABLE_RUNTIMES flag.
@DavidSpickett
Copy link
Collaborator

This problem was not caught on Windows buildbots beacuase it appeared when compiler-rt was included via LLVM_ENABLE_PROJECTS while buildbots include compiler-rt project using LLVM_ENABLE_RUNTIMES flag.

Surprising, but the bot is green so I assume LLVM_ENABLE_RUNTIMES doesn't need any fixes?

(LLVM_ENABLE_RUNTIMES is the recommended way going forward, but LLVM_ENABLE_PROJECTS is still valid and worth fixing for sure)

@omjavaid omjavaid merged commit 33e16ca into llvm:main May 2, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants