https://github.com/tstellar updated https://github.com/llvm/llvm-project/pull/125834
>From 04d55131ce724ad4a8ca2686ee54df83484f94e9 Mon Sep 17 00:00:00 2001 From: Diego Caballero <dieg0ca6all...@gmail.com> Date: Thu, 30 Jan 2025 16:09:19 -0800 Subject: [PATCH] [mlir][cmake] Add missing MLIRTestDialect dependencies This cherry picks [mlir] Fix build race condition in Pass Manager tests (d906da5ead2764579395e5006c517f2ec9afd46f) to the 20.x release branch. This addresses issues that started with https://github.com/llvm/llvm-project/pull/123910, which is already on the 20.x branch. Linaro noticed this on our flang dylib (shared library) build bot. 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" | ^~~~~~~~~~~~~~~ We have tested these changes on the buildbot for the last 2 days and had no problems. Whereas before it was failing maybe 1 in 10 builds, enough that multiple people in the community noticed it. Reported in https://github.com/llvm/llvm-project/issues/124485. --- mlir/test/lib/Pass/CMakeLists.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/mlir/test/lib/Pass/CMakeLists.txt b/mlir/test/lib/Pass/CMakeLists.txt index 6698af86b8ae66d..c5d0bab8ec74946 100644 --- a/mlir/test/lib/Pass/CMakeLists.txt +++ b/mlir/test/lib/Pass/CMakeLists.txt @@ -10,12 +10,14 @@ add_mlir_library(MLIRTestPass ADDITIONAL_HEADER_DIRS ${MLIR_MAIN_INCLUDE_DIR}/mlir/Pass + + LINK_LIBS PUBLIC + MLIRTestDialect ) mlir_target_link_libraries(MLIRTestPass PUBLIC ${conversion_libs} MLIRIR MLIRPass - MLIRTestDialect ) target_include_directories(MLIRTestPass _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
