Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LinalgExt] Scatter fusion by expansion 3/3 #19588

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

IanWood1
Copy link
Contributor

@IanWood1 IanWood1 commented Jan 2, 2025

Ignore the first 3 commits, they have their own PRs. See main issue #19091

- Update verifier to handle multiple batch dimensions for `updates` and
  `indices`.
- Refine `ScatterOp` description to align with verifier behavior and
  current usage.
  - Previous description was outdated and inconsistent with verifier
    logic.
- Restrict `dimension_map` to allow indexing only of the outermost
  dimensions, ensuring slices are inserted contiguously.
- Add new helper methods to `ScatterOp` to facilitate extended
  functionality.
- Update and add new test cases for `ScatterOp`:
  - Remove tests involving non-contiguous slices.
  - Add end-to-end tests for cases with more than one batch
    dimension.
- Fix `TilingInterfaceImpl` to support multiple "batch" dimensions
  and added test cases to `convert_to_loops.mlir` and `tiling.mlir`

Signed-off-by: Ian Wood <[email protected]>
@IanWood1 IanWood1 changed the title [LinalgExt] Scatter fusion by expansion [LinalgExt] Scatter fusion by expansion 3/3 Jan 2, 2025
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.

1 participant