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

[Codegen][SME] Recognize transposed output indexing map as transpose #18369

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

MacDue
Copy link
Member

@MacDue MacDue commented Aug 27, 2024

This restores the SME transpose lowering after #18074, which swapped around the indexing maps (making the input identity and the output a permutation).

@MacDue MacDue requested review from c-rhodes and removed request for MaheshRavishankar and hanhanW August 27, 2024 11:14
This restores the SME transpose lowering after iree-org#18074, which swapped
around the indexing maps (making the input identity and the output a
permutation).

Signed-off-by: Benjamin Maxwell <[email protected]>
Copy link
Contributor

@c-rhodes c-rhodes left a comment

Choose a reason for hiding this comment

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

LGTM cheers

@banach-space banach-space merged commit a7ed2d7 into iree-org:main Aug 30, 2024
36 of 37 checks passed
banach-space added a commit to banach-space/iree that referenced this pull request Aug 30, 2024
I've just landed an update for the affected test (see iree-org#18369), but
unfortunately forgot to re-base after the recent changes to HAL by Ben,
see iree-org#18366 and iree-org#18154.

This simply updates the test to align with the recent changes to HAL.
banach-space added a commit to banach-space/iree that referenced this pull request Aug 30, 2024
I've just landed an update for the affected test (see iree-org#18369), but
unfortunately forgot to re-base after the recent changes to HAL by Ben,
see iree-org#18366 and iree-org#18154.

This simply updates the test to align with the recent changes to HAL.

Signed-off-by: Andrzej Warzynski <[email protected]>
@banach-space
Copy link
Collaborator

Sorry, should've re-based before merging this one. Here's a fix for the test:

@MacDue MacDue deleted the sme_tr_fix branch August 30, 2024 13:51
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.

4 participants