[all-commits] [llvm/llvm-project] 42578e: [mlir][linalg] Use hasPureTensorSemantics in Trans...
Han-Chung Wang via All-commits
all-commits at lists.llvm.org
Tue Jul 1 14:15:48 PDT 2025
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 42578e8586e65fb9a3f9215d0f650eab26be92f0
https://github.com/llvm/llvm-project/commit/42578e8586e65fb9a3f9215d0f650eab26be92f0
Author: Han-Chung Wang <hanhan0912 at gmail.com>
Date: 2025-07-01 (Tue, 01 Jul 2025)
Changed paths:
M mlir/lib/Dialect/Linalg/Transforms/TransposeMatmul.cpp
Log Message:
-----------
[mlir][linalg] Use hasPureTensorSemantics in TransposeMatmul methods. (#146438)
The issue is triggered by
https://github.com/llvm/llvm-project/commit/ee070d08163ac09842d9bf0c1315f311df39faf1
that checks `TensorLikeType` when downstream projects use the pattern
without registering bufferization::BufferizationDialect. The
registration is needed because the interface implementation for builtin
types locate at `BufferizationDialect::initialize()`. However, we do not
need to fix it by the registration. The proper fix is using the linalg
method, i.e., hasPureTensorSemantics.
No additional tests are added because the functionality is well tested
in
[transpose-matmul.mlir](https://github.com/llvm/llvm-project/blob/main/mlir/test/Dialect/Linalg/transpose-matmul.mlir).
To reproduce the issue, it requires a different setup, e.g., writing a
new C++ pass, which seems not worth it.
Signed-off-by: hanhanW <hanhan0912 at gmail.com>
To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications
More information about the All-commits
mailing list