[all-commits] [llvm/llvm-project] d906da: [mlir] Fix build race condition in Pass Manager tests

[David Spickett via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d906da5ead2764579395e5006c517f2ec9afd46f
      https://github.com/llvm/llvm-project/commit/d906da5ead2764579395e5006c517f2ec9afd46f
  Author: David Spickett <david.spickett at linaro.org>
  Date:   2025-02-03 (Mon, 03 Feb 2025)

  Changed paths:
    M mlir/test/lib/Pass/CMakeLists.txt

  Log Message:
  -----------
  [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 https://github.com/llvm/llvm-project/issues/124335 and
https://github.com/llvm/llvm-project/issues/124485.

I think this was caused by https://github.com/llvm/llvm-project/pull/123910,
and so I've followed what https://github.com/llvm/llvm-project/pull/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.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications