[Mlir-commits] [mlir] [mlir][emitc] Update and extend the TOSA -> EmitC test (PR #177339)
Andrzej Warzyński
llvmlistbot at llvm.org
Thu Jan 22 12:49:29 PST 2026
banach-space wrote:
@simon-camp That's now reproduced in the CI: https://github.com/llvm/llvm-project/actions/runs/21263250658/job/61195825257?pr=177339
<img width="2136" height="186" alt="Screenshot 2026-01-22 at 20 41 55" src="https://github.com/user-attachments/assets/c10c98cf-8239-4bd9-9ca3-65ea436bcee1" />
I've managed to triage that and it's basically this extension registration:
https://github.com/llvm/llvm-project/blob/056e5a32c80d0f494797788ecdaede314eb0f947/mlir/lib/Conversion/ConvertToEmitC/ConvertToEmitCPass.cpp?plain=1#L189-L191
The test passes without that line, so I suggest removing it. I admit that I don't fully follow what _exactly_ goes wrong (other than dialect registration happens too late). This feels very subtle and it is also rather late at my end 😅
WDYT?
https://github.com/llvm/llvm-project/pull/177339
More information about the Mlir-commits
mailing list