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.

Sign up for GitHub

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

Jump to bottom

Arm64/Sve: The scaling of immediate in ldr and other instructions should take into account variable VL #100991

Open
kunalspathak opened this issue Apr 12, 2024 · 2 comments
Open

Arm64/Sve: The scaling of immediate in ldr and other instructions should take into account variable VL #100991

kunalspathak opened this issue Apr 12, 2024 · 2 comments
Assignees
@kunalspathak
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI arm-sve Work related to arm64 SVE/SVE2 support
Milestone
10.0.0

Comments

@kunalspathak
Copy link
Member

Relevant discussion: #100134 (comment) and #100743 (comment)

Today, we hardcode the VL to be 128-bits while encoding the INS_sve_ldr and INS_sve_str. We should make sure that it is VL agnostic.
@kunalspathak kunalspathak added the arm-sve Work related to arm64 SVE/SVE2 support label Apr 12, 2024
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Apr 12, 2024
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Apr 12, 2024
@kunalspathak kunalspathak self-assigned this Apr 12, 2024
@kunalspathak
Copy link
Member Author

cc: @a74nh , @tannergooding

@kunalspathak kunalspathak mentioned this issue Apr 12, 2024
Merged
@jeffschwMSFT jeffschwMSFT added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Apr 15, 2024
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

@vcsjones vcsjones removed the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Apr 16, 2024
@JulieLeeMSFT JulieLeeMSFT added this to the 9.0.0 milestone Apr 19, 2024
@JulieLeeMSFT JulieLeeMSFT removed the untriaged New issue has not been triaged by the area owner label Apr 19, 2024
@kunalspathak kunalspathak modified the milestones: 9.0.0, 10.0.0 Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kunalspathak kunalspathak

Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI arm-sve Work related to arm64 SVE/SVE2 support
Projects
None yet
Milestone
10.0.0
Development

No branches or pull requests
4 participants
@vcsjones @jeffschwMSFT @kunalspathak @JulieLeeMSFT