Skip to content

Commit d906da5

Browse files
committed
[mlir] Fix build race condition in Pass Manager tests
That Linaro has been seeing on our dylib bot: https://lab.llvm.org/staging/#/builders/171/builds/79 In file included from /home/tcwg-buildbot/worker/flang-aarch64-dylib/llvm-project/mlir/test/lib/Pass/TestPassManager.cpp:10: /home/tcwg-buildbot/worker/flang-aarch64-dylib/llvm-project/mlir/test/lib/Pass/../Dialect/Test/TestOps.h:148:10: fatal error: 'TestOps.h.inc' file not found 148 | #include "TestOps.h.inc" | ^~~~~~~~~~~~~~~ Reported in llvm#124335 and llvm#124485. I think this was caused by llvm#123910, and so I've followed what llvm#125004 did, which seems to be a follow up to that. Same kind of error. I was not able to reproduce the failure locally, but dylib and normal builds are ok with this change so I will push it and monitor the bot's results for a few days.
1 parent 9216419 commit d906da5

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

mlir/test/lib/Pass/CMakeLists.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,15 @@ add_mlir_library(MLIRTestPass
1111

1212
ADDITIONAL_HEADER_DIRS
1313
${MLIR_MAIN_INCLUDE_DIR}/mlir/Pass
14+
15+
LINK_LIBS PUBLIC
16+
MLIRTestDialect
1417
)
1518
mlir_target_link_libraries(MLIRTestPass PUBLIC
1619
${conversion_libs}
1720
MLIRIR
1821
MLIRSPIRVTransforms
1922
MLIRPass
20-
MLIRTestDialect
2123
)
2224

2325
target_include_directories(MLIRTestPass

0 commit comments

Comments
 (0)