mlir: fatal error: 'TestOps.h.inc' file not found on linux s390x
#124335
Labels
Comments
|
@llvm/issue-subscribers-mlir Author: Sylvestre Ledru (sylvestre)
log: https://llvm-jenkins.debian.net/job/llvm-toolchain-jammy-binaries/architecture=s390x,distribution=jammy,label=s390x/1911/console
|
|
See also #124485, it's happening only some of the time on our build bot. |
DavidSpickett
added a commit
that referenced
this issue
Feb 3, 2025
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 #124335 and #124485. I think this was caused by #123910, and so I've followed what #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.
github-actions bot
pushed a commit
to arm/arm-toolchain
that referenced
this issue
Feb 3, 2025
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/llvm-project#124335 and llvm/llvm-project#124485. I think this was caused by llvm/llvm-project#123910, and so I've followed what llvm/llvm-project#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.
|
Based on the file name in the error, I think 72b73c9 fixed this. I'm about to cherry pick that to 20.x. |
DavidSpickett
pushed a commit
to DavidSpickett/llvm-project
that referenced
this issue
Feb 5, 2025
This cherry picks [mlir][cmake] Add missing MLIRTestDialect dependency (llvm#125004) (72b73c9) and [mlir] Fix build race condition in Pass Manager tests (d906da5) to the 20.x release branch. Both of these are addressing issues that started with llvm#123910, which is already on the 20.x branch. In each case the cherry-picked changes correct problems that would otherwise be a race condition in certain builds. Linaro noticed this on our flang dylib (shared library) build bot. Failures look like: 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 llvm#124485 and llvm#124335. Co-authored-by: Diego Caballero <[email protected]>
Icohedron
pushed a commit
to Icohedron/llvm-project
that referenced
this issue
Feb 11, 2025
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.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
log: https://llvm-jenkins.debian.net/job/llvm-toolchain-jammy-binaries/architecture=s390x,distribution=jammy,label=s390x/1911/console
Recent regression
range:
3274bf6..c3b40c7
The text was updated successfully, but these errors were encountered: