-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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.
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
JIT: Rewrite register parameter homing #100572
Merged
Merged
Changes from 24 commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
40b32cd
JIT: Rewrite register parameter homing
jakobbotsch bf43089
Remove more unnecessary stuff
jakobbotsch 803b86c
Clean up
jakobbotsch 843c1c8
Handle by register instead of edge
jakobbotsch 0b445fb
Handle circularity v1
jakobbotsch b04c7a1
Simplify, remove args when no longer busy
jakobbotsch d3ea5d3
Handle init reg properly
jakobbotsch e3ba587
Merge branch 'main' of github.com:dotnet/runtime into register-homing
jakobbotsch fb3a7fe
Add stress mode
jakobbotsch 7926f1e
Fix after merge
jakobbotsch 705090d
Enable it always for CI run
jakobbotsch 2a49d31
Merge branch 'main' of github.com:dotnet/runtime into register-homing
jakobbotsch fcaa40a
Fix after merge
jakobbotsch 0b04efc
Make multiple edges for arm32 doubles
jakobbotsch 0e20159
Simplify
jakobbotsch 753bdd5
Further simplification
jakobbotsch 993e883
Run jit-format
jakobbotsch 85fd667
Fix release build
jakobbotsch 196c7d7
Fix RISC-V build
jakobbotsch f271cc5
Move new stress logic into its own function in LSRA
jakobbotsch 9d7bbbe
Disable stress mode by default
jakobbotsch 22acb2a
Fix build, hopefully last cleanup
jakobbotsch 3355002
Add function headers and fix a comment
jakobbotsch bbdc3d0
Add arm32 double move optimization
jakobbotsch 6ab6749
Rename FindNodeToHandle -> FindNodeToProcess
jakobbotsch 865bd4d
Merge branch 'main' of github.com:dotnet/runtime into register-homing
jakobbotsch 75126d7
Switch lvaParameterPassingInfo accesses to lvaGetParameterABIInfo
jakobbotsch File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The bool could be a return value, but seems ok this way too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also most functions are using this pattern so it seems good to be consistent.