[Mlir-commits] [mlir] [mlir][TOSA] Fix linalg lowering of depthwise conv2d (PR #130282)

Fri Mar 7 05:32:38 PST 2025

llvm-ci wrote:

LLVM Buildbot has detected a new failure on builder `mlir-nvidia-gcc7` running on `mlir-nvidia` while building `mlir` at step 7 "test-build-check-mlir-build-only-check-mlir".

Full details are available at: https://lab.llvm.org/buildbot/#/builders/116/builds/11211

<details>
<summary>Here is the relevant piece of the build log for the reference</summary>

```
Step 7 (test-build-check-mlir-build-only-check-mlir) failure: test (failure)
******************** TEST 'MLIR :: Conversion/TosaToLinalg/tosa-to-linalg-named.mlir' FAILED ********************
Exit Code: 1

Command Output (stdout):
--
# RUN: at line 1
/vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/mlir-opt --verify-each --split-input-file -pass-pipeline="builtin.module(func.func(tosa-to-linalg-named))" /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Conversion/TosaToLinalg/tosa-to-linalg-named.mlir -verify-diagnostics -o -| /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/FileCheck /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Conversion/TosaToLinalg/tosa-to-linalg-named.mlir
# executed command: /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/mlir-opt --verify-each --split-input-file '-pass-pipeline=builtin.module(func.func(tosa-to-linalg-named))' /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Conversion/TosaToLinalg/tosa-to-linalg-named.mlir -verify-diagnostics -o -
# .---command stderr------------
# | <unknown>:0: error: unexpected error: expected integer or index type
# | mlir-opt: /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/include/mlir/IR/StorageUniquerSupport.h:180: static ConcreteT mlir::detail::StorageUserBase<ConcreteT, BaseT, StorageT, UniquerT, Traits>::get(mlir::MLIRContext*, Args&& ...) [with Args = {mlir::Type&, const llvm::APInt&}; ConcreteT = mlir::IntegerAttr; BaseT = mlir::Attribute; StorageT = mlir::detail::IntegerAttrStorage; UniquerT = mlir::detail::AttributeUniquer; Traits = {mlir::TypedAttr::Trait}]: Assertion `succeeded( ConcreteT::verifyInvariants(getDefaultDiagnosticEmitFn(ctx), args...))' failed.
# | PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace.
# | Stack dump:
# | 0.	Program arguments: /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/mlir-opt --verify-each --split-input-file -pass-pipeline=builtin.module(func.func(tosa-to-linalg-named)) /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Conversion/TosaToLinalg/tosa-to-linalg-named.mlir -verify-diagnostics -o -
# | Stack dump without symbol names (ensure you have llvm-symbolizer in your PATH or set the environment var `LLVM_SYMBOLIZER_PATH` to point to it):
# | 0  mlir-opt  0x000058604024e7bb llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) + 59
# | 1  mlir-opt  0x000058604024bb84
# | 2  libc.so.6 0x00007daa5011a520
# | 3  libc.so.6 0x00007daa5016e9fc pthread_kill + 300
# | 4  libc.so.6 0x00007daa5011a476 raise + 22
# | 5  libc.so.6 0x00007daa501007f3 abort + 211
# | 6  libc.so.6 0x00007daa5010071b
# | 7  libc.so.6 0x00007daa50111e96
# | 8  mlir-opt  0x000058604328b86c mlir::IntegerAttr::get(mlir::Type, llvm::APInt const&) + 620
# | 9  mlir-opt  0x000058604326afef mlir::Builder::getIntegerAttr(mlir::Type, long) + 223
# | 10 mlir-opt  0x0000586042f81eed
# | 11 mlir-opt  0x0000586042f70743
# | 12 mlir-opt  0x0000586042f7200c
# | 13 mlir-opt  0x00005860431bee5b mlir::ConversionPattern::matchAndRewrite(mlir::Operation*, mlir::PatternRewriter&) const + 587
# | 14 mlir-opt  0x0000586046030bdb mlir::PatternApplicator::matchAndRewrite(mlir::Operation*, mlir::PatternRewriter&, llvm::function_ref<bool (mlir::Pattern const&)>, llvm::function_ref<void (mlir::Pattern const&)>, llvm::function_ref<llvm::LogicalResult (mlir::Pattern const&)>) + 1931
# | 15 mlir-opt  0x00005860431c2724
# | 16 mlir-opt  0x00005860431c2fa8 mlir::OperationConverter::convert(mlir::ConversionPatternRewriter&, mlir::Operation*) + 56
# | 17 mlir-opt  0x00005860431c5794 mlir::OperationConverter::convertOperations(llvm::ArrayRef<mlir::Operation*>) + 724
# | 18 mlir-opt  0x00005860431c7bf4 mlir::applyFullConversion(mlir::Operation*, mlir::ConversionTarget const&, mlir::FrozenRewritePatternSet const&, mlir::ConversionConfig) + 148
# | 19 mlir-opt  0x0000586042f6cdad
# | 20 mlir-opt  0x000058604311033e mlir::detail::OpToOpPassAdaptor::run(mlir::Pass*, mlir::Operation*, mlir::AnalysisManager, bool, unsigned int) + 1294
# | 21 mlir-opt  0x000058604311069a mlir::detail::OpToOpPassAdaptor::runPipeline(mlir::OpPassManager&, mlir::Operation*, mlir::AnalysisManager, bool, unsigned int, mlir::PassInstrumentor*, mlir::PassInstrumentation::PipelineParentInfo const*) + 250
# | 22 mlir-opt  0x0000586043110a92
# | 23 mlir-opt  0x000058604310f8cd mlir::detail::OpToOpPassAdaptor::runOnOperationAsyncImpl(bool) + 3165
# | 24 mlir-opt  0x000058604311019a mlir::detail::OpToOpPassAdaptor::run(mlir::Pass*, mlir::Operation*, mlir::AnalysisManager, bool, unsigned int) + 874
# | 25 mlir-opt  0x0000586043111243
# | 26 mlir-opt  0x000058604311196f mlir::PassManager::run(mlir::Operation*) + 1295
# | 27 mlir-opt  0x00005860431024a7
# | 28 mlir-opt  0x0000586043102d3c
# | 29 mlir-opt  0x0000586043103091
# | 30 mlir-opt  0x000058604322676d
# | 31 mlir-opt  0x000058604322733e mlir::splitAndProcessBuffer(std::unique_ptr<llvm::MemoryBuffer, std::default_delete<llvm::MemoryBuffer>>, llvm::function_ref<llvm::LogicalResult (std::unique_ptr<llvm::MemoryBuffer, std::default_delete<llvm::MemoryBuffer>>, llvm::raw_ostream&)>, llvm::raw_ostream&, llvm::StringRef, llvm::StringRef) + 1806
# | 32 mlir-opt  0x00005860430fc18c mlir::MlirOptMain(llvm::raw_ostream&, std::unique_ptr<llvm::MemoryBuffer, std::default_delete<llvm::MemoryBuffer>>, mlir::DialectRegistry&, mlir::MlirOptMainConfig const&) + 220
# | 33 mlir-opt  0x00005860431031ef mlir::MlirOptMain(int, char**, llvm::StringRef, llvm::StringRef, mlir::DialectRegistry&) + 287
# | 34 mlir-opt  0x00005860431036ae mlir::MlirOptMain(int, char**, llvm::StringRef, mlir::DialectRegistry&) + 414
...

```

</details>

https://github.com/llvm/llvm-project/pull/130282