Skip to content

Commit 5ddbf66

Browse files
committed
[mlir] [test] Do not add dependencies on llvm tools in standalone builds
Since LLVM tools are installed system-wide, adding dependencies on them is unnecessary. Furthermore, it is problematic for multilib builds, where the tools are only built once, for the native ABI, and therefore are not listed in CMake files for non-native ABIs.
1 parent f96337e commit 5ddbf66

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

mlir/test/CMakeLists.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,6 @@ configure_lit_site_cfg(
9999
)
100100

101101
set(MLIR_TEST_DEPENDS
102-
FileCheck count not split-file
103102
mlir-capi-ir-test
104103
mlir-capi-irdl-test
105104
mlir-capi-llvm-test
@@ -121,6 +120,9 @@ set(MLIR_TEST_DEPENDS
121120
tblgen-lsp-server
122121
tblgen-to-irdl
123122
)
123+
if(NOT MLIR_STANDALONE_BUILD)
124+
list(APPEND MLIR_TEST_DEPENDS FileCheck count not split-file)
125+
endif()
124126

125127
set(MLIR_TEST_DEPENDS ${MLIR_TEST_DEPENDS}
126128
mlir-capi-pdl-test

0 commit comments

Comments
 (0)