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

Reland "[DispatchCreation] Run preprocessing before..." #18939

Merged

Conversation

IanWood1
Copy link
Contributor

@IanWood1 IanWood1 commented Oct 29, 2024

Land #18934 after fixing mi250 regressions. Instead of moving the whole pass (which "interchanges" the indexing maps and was the root of the regressions), only move the single pattern GatherFusionPattern into ElementwiseOpFusion.cpp.

closes: #19077

@IanWood1 IanWood1 force-pushed the revert-18934-revert-18920-run_preprocessing_before_elem_fusion branch 4 times, most recently from 459313b to a301118 Compare November 13, 2024 20:56
@IanWood1 IanWood1 marked this pull request as ready for review November 13, 2024 20:58
@IanWood1
Copy link
Contributor Author

@MaheshRavishankar or @hanhanW can I get a review on this please?

@IanWood1
Copy link
Contributor Author

Thank you!

@IanWood1 IanWood1 merged commit 540cebf into main Nov 18, 2024
39 checks passed
@IanWood1 IanWood1 deleted the revert-18934-revert-18920-run_preprocessing_before_elem_fusion branch November 18, 2024 22:55
Groverkss pushed a commit to Groverkss/iree that referenced this pull request Dec 1, 2024
Land iree-org#18934 after fixing mi250 regressions. Instead of
moving the whole pass (which "interchanges" the indexing maps and was
the root of the regressions), only move the single pattern
`GatherFusionPattern` into `ElementwiseOpFusion.cpp`.


closes: iree-org#19077

Signed-off-by: Ian Wood <[email protected]>
giacs-epic pushed a commit to giacs-epic/iree that referenced this pull request Dec 4, 2024
Land iree-org#18934 after fixing mi250 regressions. Instead of
moving the whole pass (which "interchanges" the indexing maps and was
the root of the regressions), only move the single pattern
`GatherFusionPattern` into `ElementwiseOpFusion.cpp`.

closes: iree-org#19077

Signed-off-by: Ian Wood <[email protected]>
Signed-off-by: Giacomo Serafini <[email protected]>
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.

Op uses negative memory exceeding 65536 byte limit
2 participants