Skip to content

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

Merged
omjavaid merged 2 commits intollvm:mainfrom
omjavaid:main
May 2, 2024
Merged

[compiler-rt] Fix CMake configure on Windows#90843
omjavaid merged 2 commits intollvm:mainfrom
omjavaid:main

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)

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.

3 participants