Skip to content

Conversation

@pawosm-arm
Copy link
Contributor

@pawosm-arm pawosm-arm commented Jun 24, 2025

This PR contains two commits:

Downstream issue: #218

pawosm-arm and others added 2 commits June 23, 2025 16:11
…etween fixed and scalable vectors (#219)"

This reverts commit e99c0aa.
…ectors. (#130973)

For function whose vscale_range is limited to a single value we can size
scalable vectors. This aids SROA by allowing scalable vector load and
store operations to be considered for replacement whereby bitcasts
through memory can be replaced by vector insert or extract operations.
@voltur01
Copy link
Contributor

I assume the new change is not on the upstream 20.x branch and we cannot add it there, thus we still need the downstream change. The only comment is then we need to still track this as a downstream change, similarly to the reverted commit.

Copy link
Contributor

@paulwalker-arm paulwalker-arm left a comment

Choose a reason for hiding this comment

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

The code changes look consistent to me.

@pawosm-arm pawosm-arm changed the title Backport of [LLVM][SROA] Teach SROA how to "bitcast" between fixed and scalable vectors. [Downstream change] Backport of [LLVM][SROA] Teach SROA how to "bitcast" between fixed and scalable vectors. Jun 25, 2025
@pawosm-arm
Copy link
Contributor Author

I assume the new change is not on the upstream 20.x branch and we cannot add it there, thus we still need the downstream change. The only comment is then we need to still track this as a downstream change, similarly to the reverted commit.

I've updated the title and the initial comment.

Copy link
Contributor

@voltur01 voltur01 left a comment

Choose a reason for hiding this comment

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

Thank you!

@pratlucas pratlucas merged commit 9d03ad2 into release/arm-software/20.x Jun 25, 2025
@pratlucas pratlucas deleted the users/pawosm-arm/release/arm-software/20.x/llvm-sroa-backport branch June 25, 2025 14:35
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.

5 participants