[Mlir-commits] [mlir] Fix support for complex types in `transform.structured.pad` (PR #139841)

Andrzej Warzyński llvmlistbot at llvm.org
Tue May 20 08:34:20 PDT 2025


================

----------------
banach-space wrote:

Rather than adding a completely new test case, could you re-use or modify one of the existing ones? Otherwise, it’s harder to see what makes the new test "special" or "unique" - modifying an existing test often makes it easier to highlight the impact of a change.

That said, I don’t see any truly “basic” cases for padding linalg.matmul, so it’s reasonable if you need to create your own. Alternatively, you might be able to re-use one of the `pad_to_multiple_of` tests. I'm not very familiar with `transform.structured.pad`, but do you actually rely on `pad_to_multiple_of` to verify this change?

Separately, please make sure your test is self-documenting - for example, consider renaming `@pad_matmul` to something more descriptive. You can find relevant testing guidelines here:

* https://mlir.llvm.org/getting_started/TestingGuide/#contributor-guidelines

Thanks!

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


More information about the Mlir-commits mailing list