Skip to content

Conversation

@jakobbotsch
Copy link
Member

Parameters are not passed in registers for OSR methods, so preferencing to certain registers based on this information does not make sense.

Follow-up to #110819

Parameters are not passed in registers for OSR methods, so preferencing
to certain registers based on this information does not make sense.
@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Feb 11, 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.

@jakobbotsch jakobbotsch marked this pull request as ready for review February 12, 2025 20:20
Copilot AI review requested due to automatic review settings February 12, 2025 20:20
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 wasn't able to review any files in this pull request.

Files not reviewed (4)
  • src/coreclr/jit/codegencommon.cpp: Language not supported
  • src/coreclr/jit/lower.cpp: Language not supported
  • src/coreclr/jit/lsra.cpp: Language not supported
  • src/coreclr/jit/lsrabuild.cpp: Language not supported

@jakobbotsch
Copy link
Member Author

cc @dotnet/jit-contrib PTAL @kunalspathak

Minor diffs

Copy link
Contributor

@kunalspathak kunalspathak left a comment

Choose a reason for hiding this comment

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

LGTM

@jakobbotsch jakobbotsch merged commit e9a91f9 into dotnet:main Feb 13, 2025
109 of 111 checks passed
@jakobbotsch jakobbotsch deleted the osr-no-initial-reg-ref branch February 13, 2025 10:06
@github-actions github-actions bot locked and limited conversation to collaborators Mar 16, 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.

2 participants