Skip to content

[CPU] Improve TileRootAndFuseProducerConsumer like TileAndFuse pass.#21351

Merged
hanhanW merged 1 commit intoiree-org:mainfrom
hanhanW:improve-tile-root-and-fuse
Jul 14, 2025
Merged

[CPU] Improve TileRootAndFuseProducerConsumer like TileAndFuse pass.#21351
hanhanW merged 1 commit intoiree-org:mainfrom
hanhanW:improve-tile-root-and-fuse

Conversation

@hanhanW
Copy link
Copy Markdown
Contributor

@hanhanW hanhanW commented Jul 11, 2025

Three major changes:

  • Support scalable tile sizes.
  • Do not signal a failure when a consumer is not fusible. Instead, it leaves it outside the loop.
  • Port tile_and_fuse.mlir test cases to the lit test.

The emitOpError in TileAndFuseUtils.cpp is removed because it is expected to let callers handle the failure. Callers may not always want the error message.

Two major changes:

- Support scalable tile sizes.
- Do not signal a failure when a consumer is not fusible. Instead, it
  leaves it outside the loop.

Signed-off-by: hanhanW <hanhan0912@gmail.com>
Copy link
Copy Markdown
Contributor

@pashu123 pashu123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hanhanW hanhanW merged commit aee9ec1 into iree-org:main Jul 14, 2025
44 checks passed
@hanhanW hanhanW deleted the improve-tile-root-and-fuse branch July 14, 2025 17:15
AWoloszyn pushed a commit that referenced this pull request Dec 1, 2025
…21351)

Three major changes:

- Support scalable tile sizes.
- Do not signal a failure when a consumer is not fusible. Instead, it
leaves it outside the loop.
- Port tile_and_fuse.mlir test cases to the lit test.

The `emitOpError` in `TileAndFuseUtils.cpp` is removed because it is
expected to let callers handle the failure. Callers may not always want
the error message.

Signed-off-by: hanhanW <hanhan0912@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants