Skip to content

Conversation

@jakobbotsch
Copy link
Member

@jakobbotsch jakobbotsch commented Dec 20, 2024

The normal homing is general enough now to handle the registers mapping to the special Swift parameters.

Fix #110808

Prerequisite for #110819

The normal homing is general enough now to handle the Swift parameter
registers.
@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Dec 20, 2024
@jakobbotsch jakobbotsch marked this pull request as ready for review December 27, 2024 23:01
@jakobbotsch
Copy link
Member Author

PTAL @dotnet/jit-contrib

Minor diffs where some register homing gets reordered for these Swift functions.

@jakobbotsch jakobbotsch requested a review from a team December 27, 2024 23:02
@jakobbotsch
Copy link
Member Author

/azp run runtime-coreclr superpmi-diffs

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jakobbotsch
Copy link
Member Author

@kunalspathak Can you PTAL?

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 951ec77 into dotnet:main Jan 8, 2025
108 checks passed
@jakobbotsch jakobbotsch deleted the unify-swift-param-register-homing branch January 8, 2025 10:22
@github-actions github-actions bot locked and limited conversation to collaborators Feb 8, 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.

JIT: Swift register homing can be unified with general register homing

2 participants