Skip to content

[Transform][RemoveSingleElemVector] Add support for vector.store lowering pattern for single-elem vector.#1011

Merged
mshahneo merged 1 commit intointel:mainfrom
mshahneo:FSW-12505_for_upstream
Jan 28, 2025
Merged

[Transform][RemoveSingleElemVector] Add support for vector.store lowering pattern for single-elem vector.#1011
mshahneo merged 1 commit intointel:mainfrom
mshahneo:FSW-12505_for_upstream

Conversation

@mshahneo
Copy link
Copy Markdown
Contributor

@mshahneo mshahneo commented Jan 28, 2025

SPIR-V does not support single element vector, hence vector.store with single element vector needs to be transformed in way suitable for SPIR-V conversion.

Since, the single element vector needs to be converted to a scalar and vector.store does not support scalar, replacing vector.store with memref.store.

Please review these guidelines to help with the review process:

  • Have you provided a meaningful PR description?
  • Have you added a test, a reproducer, or a reference to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • If this PR is a work in progress, are you filing the PR as a draft?
  • Have you organized your commits logically and ensured each can be built by itself?

@mshahneo mshahneo changed the title [Transform][RemoveSingleElemVector] Add support for vector.store lowe… [Transform][RemoveSingleElemVector] Add support for vector.store lowering pattern for single-elem vector. Jan 28, 2025
@mshahneo mshahneo requested a review from Garra1980 January 28, 2025 18:01
…ring pattern for single-elem vector.

SPIR-V does not support single element vector, hence vector.store with single element vector
needs to be transformed in way suitable for SPIR-V conversion.

Since, the single element vector needs to be converted to a scalar and vector.store does not
support scalar, replacing vector.store with memref.store.
@mshahneo mshahneo force-pushed the FSW-12505_for_upstream branch from eae6f1e to e159885 Compare January 28, 2025 22:27
@mshahneo mshahneo merged commit 3269717 into intel:main Jan 28, 2025
2 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