Skip to content

Conversation

@pawosm-arm
Copy link
Contributor

It's a cherry-pick from the sve-vls-sroa branch of Paul Walker's LLVM fork.

Ongoing upstream attempt:
llvm/llvm-project#130973

Downstream issue: #218

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.

…ixed and scalable vectors.

It's a cherry-pick from the sve-vls-sroa branch of Paul Walker's LLVM fork.

Ongoing upstream attempt:
    llvm/llvm-project#130973

Downstream issue: #218

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.
@pawosm-arm
Copy link
Contributor Author

@huntergr-arm or @davemgreen can I ask you for a comment which other approvers could refer to? I'm lacking candidates with domain knowledge to make it land....

@kiranchandramohan
Copy link
Contributor

Did the cherry-pick happen cleanly?

@pawosm-arm
Copy link
Contributor Author

Did the cherry-pick happen cleanly?

Yes, not a single conflict.

@paulwalker-arm
Copy link
Contributor

paulwalker-arm commented Mar 21, 2025

Did the cherry-pick happen cleanly?

Yes, not a single conflict.

I can confirm this PR matches my original implementation and is safe to land.

Copy link
Contributor

@kiranchandramohan kiranchandramohan left a comment

Choose a reason for hiding this comment

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

LGTM.

@pawosm-arm pawosm-arm merged commit e99c0aa into release/arm-software/20.x Mar 21, 2025
@pawosm-arm pawosm-arm deleted the users/pawosm-arm/release/arm-software/20.x/atfl-sve-vls-sroa branch March 21, 2025 11:03
pawosm-arm added a commit that referenced this pull request Jun 24, 2025
…etween fixed and scalable vectors (#219)"

This reverts commit e99c0aa.
pratlucas pushed a commit that referenced this pull request Jun 25, 2025
…st" between fixed and scalable vectors. (#399)

This PR contains two commits:

- revert of "[Downstream change][LLVM][SROA] Teach SROA how to "bitcast"
between fixed and scalable vectors (#219)"
- backport of usprtream's "[LLVM][SROA] Teach SROA how to "bitcast"
between fixed and scalable vectors."

Downstream issue: #218

---------

Co-authored-by: Paul Walker <[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.

4 participants