-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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: Add a pass of liveness for the new locals inside physical promotion #86043
Merged
jakobbotsch
merged 18 commits into
dotnet:main
from
jakobbotsch:physical-promotion-liveness
May 19, 2023
Merged
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
5d676a2
JIT: Physical promotion liveness
jakobbotsch e7c2805
Compiler fixes
jakobbotsch 92d28a6
Fix build
jakobbotsch dbd223d
Stretch 2
jakobbotsch 3e1ef9a
Run jit-format
jakobbotsch 8e688cd
Merge branch 'main' of github.com:dotnet/runtime into physical-promot…
jakobbotsch 1c7e184
Fix
jakobbotsch cb8ea30
Fixes, run jit-format
jakobbotsch 68b822b
Handle non-remainder uses of struct local
jakobbotsch 6798af5
Cleanup; run jit-format
jakobbotsch 70c65b3
Handle QMARKs
jakobbotsch 6a68b10
Oopsie
jakobbotsch 585fbb6
More JITDUMPing, mark more locals as dying
jakobbotsch 3225eb9
Merge branch 'main' of github.com:dotnet/runtime into physical-promot…
jakobbotsch 8d73b0e
Revert, run jit-format
jakobbotsch 5f0f264
Fix some logging
jakobbotsch c78f76e
Fix build
jakobbotsch 361ab86
Rename, fix some comments
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
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.
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.
This is just used to print "(last use)" in the dump outside the new locals introduced by physical promotion.