Skip to content

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