[all-commits] [llvm/llvm-project] 5f8e7e: [mlir][linalg][nfc] Split hoisting tests into dedi...
Andrzej Warzyński via All-commits
all-commits at lists.llvm.org
Wed Jun 25 11:33:00 PDT 2025
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 5f8e7ed5a38c79f67e79ccc6b57454e6f38a758a
https://github.com/llvm/llvm-project/commit/5f8e7ed5a38c79f67e79ccc6b57454e6f38a758a
Author: Andrzej Warzyński <andrzej.warzynski at arm.com>
Date: 2025-06-25 (Wed, 25 Jun 2025)
Changed paths:
M mlir/test/Dialect/Linalg/hoisting.mlir
Log Message:
-----------
[mlir][linalg][nfc] Split hoisting tests into dedicated test functions (#145234)
Refactors the `@hoist_vector_transfer_pairs` test function in
`hoisting.mlir` into smaller, focused test functions - each covering a
specific `vector.transfer_read`/`vector.transfer_write` pair.
This makes it easier to identify which edge cases are tested, spot
duplication, and write more targeted and readable check lines, with less
surrounding noise.
This refactor also helped identify some issues with the original
`@hoist_vector_transfer_pairs` test:
* Input variables `%val` and `%cmp` were unused.
* There were no check lines for reads from `memref5`.
**Note for reviewers (current and future):**
This PR is split into small, incremental, and self-contained commits. It
should be easier to follow the changes by reviewing those commits
individually, rather than reading the full squashed diff. However, this
will be merged as a single commit to avoid adding unnecessary history
noise in-tree.
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