[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