Skip to content

SVE: Slightly better calculation of stack frame for ldr/str#114284

Merged
kunalspathak merged 4 commits intodotnet:mainfrom
kunalspathak:sve-frame-address
Apr 9, 2025
Merged

SVE: Slightly better calculation of stack frame for ldr/str#114284
kunalspathak merged 4 commits intodotnet:mainfrom
kunalspathak:sve-frame-address

Conversation

@kunalspathak
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 4, 2025 20:58
@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Apr 4, 2025
@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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/coreclr/jit/emitarm64.cpp:8334

  • The associated comment indicates handling large immediates (> 9 bits), but the condition uses the threshold 4096. Consider aligning the comment with the code logic to avoid confusion.
if (imm < 4096)

@kunalspathak
Copy link
Contributor Author

/azp run Fuzzlyn, Antigen

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@kunalspathak
Copy link
Contributor Author

Antigen and Fuzzlyn failures are unrelated.

@dotnet/jit-contrib PTAL

@kunalspathak
Copy link
Contributor Author

ping @dotnet/jit-contrib @dotnet/arm64-contrib

Copy link
Member

@jakobbotsch jakobbotsch left a comment

Choose a reason for hiding this comment

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

LGTM with a suggestion

@kunalspathak
Copy link
Contributor Author

/ba-g failures are timeout related

@kunalspathak kunalspathak merged commit 376e4f9 into dotnet:main Apr 9, 2025
106 of 111 checks passed
@kunalspathak kunalspathak deleted the sve-frame-address branch April 9, 2025 03:55
@github-actions github-actions bot locked and limited conversation to collaborators May 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants