-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Implement Swift lowering algorithm in the CoreCLR type system #99438
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
Merged
jkoritzinsky
merged 14 commits into
dotnet:main
from
jkoritzinsky:swift-lowering-coreclr
Mar 12, 2024
Merged
Changes from 9 commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
5e38fd4
Start implementing the algorithm in the CoreCLR type system.
jkoritzinsky fc93eb4
First pass implementing the CoreCLR side of the Swift physical lowering.
jkoritzinsky 869e6d0
Add InlineArray support and compile on all platforms (needed for altjit)
jkoritzinsky fc47d41
Fix interval merging algorithm to correctly split sequential interval…
jkoritzinsky 171a75f
Fix the final lowering step to loop over the interval array
jkoritzinsky 123750b
Additional fixes to the Swift algorithm.
jkoritzinsky 478d7be
Add missing case
jkoritzinsky 60b07c8
Merge branch 'main' of github.com:dotnet/runtime into swift-lowering-…
jkoritzinsky 5a2126c
Move increment in the NativeAOT implementation
jkoritzinsky 9e53a9e
Add contracts and PR feedback
jkoritzinsky 581a50a
Remove refs and fix spelling
jkoritzinsky 3254d51
PR feedback
jkoritzinsky e586ebe
Add back underlying type and add comment with rationale
jkoritzinsky e6fd1af
Refactor out some constants
jkoritzinsky 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 hidden or 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 hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.