-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[Fuchsia] Not building llvm-mt when LIBXML2 is not enabled. #135877
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
Conversation
This patch prevents including llvm-mt to LLVM_TOOLCHAIN_TOOLS when LIBXML2 is not explicitly enabled.
@llvm/pr-subscribers-clang Author: Haowei (zeroomega) ChangesThis patch prevents including llvm-mt to LLVM_TOOLCHAIN_TOOLS when LIBXML2 is not explicitly enabled. Full diff: https://github.com/llvm/llvm-project/pull/135877.diff 1 Files Affected:
diff --git a/clang/cmake/caches/Fuchsia-stage2.cmake b/clang/cmake/caches/Fuchsia-stage2.cmake
index 99890b8246ad7..e10855f5ef31b 100644
--- a/clang/cmake/caches/Fuchsia-stage2.cmake
+++ b/clang/cmake/caches/Fuchsia-stage2.cmake
@@ -459,7 +459,6 @@ set(LLVM_TOOLCHAIN_TOOLS
llvm-libtool-darwin
llvm-lipo
llvm-ml
- llvm-mt
llvm-nm
llvm-objcopy
llvm-objdump
@@ -481,6 +480,10 @@ set(LLVM_TOOLCHAIN_TOOLS
scan-build-py
CACHE STRING "")
+if (LLVM_ENABLE_LIBXML2)
+ list(APPEND LLVM_TOOLCHAIN_TOOLS llvm-mt)
+endif()
+
set(LLVM_Toolchain_DISTRIBUTION_COMPONENTS
bolt
clang
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/59/builds/16088 Here is the relevant piece of the build log for the reference
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/18/builds/14550 Here is the relevant piece of the build log for the reference
|
…136216) Reverts #135877 This is causing some problems on Fuchsia's windows CI. We'll need a different solution to triage other builders. https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-windows-x64/b8717476961063994817/overview
… enabled." (#136216) Reverts llvm/llvm-project#135877 This is causing some problems on Fuchsia's windows CI. We'll need a different solution to triage other builders. https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-windows-x64/b8717476961063994817/overview
…lvm#136216) Reverts llvm#135877 This is causing some problems on Fuchsia's windows CI. We'll need a different solution to triage other builders. https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-windows-x64/b8717476961063994817/overview
…lvm#136216) Reverts llvm#135877 This is causing some problems on Fuchsia's windows CI. We'll need a different solution to triage other builders. https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-windows-x64/b8717476961063994817/overview
This patch prevents including llvm-mt to LLVM_TOOLCHAIN_TOOLS when LIBXML2 is not explicitly enabled.