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

[VectorExt] Add support for masking for toLayout vectorization #19728

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

manupak
Copy link
Contributor

@manupak manupak commented Jan 17, 2025

Currently, when we vectorize toLayout ops it expects the source to be statically shaped. However, if we are to vectorize dynamic shaped tensor to static shaped vector, toLayout op needs to perform masking.

This commit adds support to add masking in the process of vectorization of toLayout op.

@manupak manupak requested a review from Groverkss January 17, 2025 12:30
@manupak manupak force-pushed the vector-ext-vectorize-masking branch 3 times, most recently from b55abc9 to 00b7184 Compare January 23, 2025 12:55
@@ -0,0 +1,30 @@
# Copyright 2024 The IREE Authors
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: 2025

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@manupak manupak force-pushed the vector-ext-vectorize-masking branch from 00b7184 to 891120a Compare January 23, 2025 15:15
Currently, when we vectorize toLayout ops it expects the
source to be statically shaped. However, if we are to vectorize
dynamic shaped tensor to static shaped vector, toLayout op
needs to perform masking.

This commit adds support to add masking in the process of
vectorization of toLayout op.

Signed-off-by: Manupa Karunaratne <[email protected]>
@manupak manupak merged commit 9b26aef into iree-org:main Jan 23, 2025
40 checks passed
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