-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[IRGen] Generate compressed representation of value witnesses #63813
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
drexin
merged 29 commits into
swiftlang:main
from
drexin:wip-bytecode-witnesses-cleanup
Feb 24, 2023
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
340a4d7
WIP: Store layout string in type metadata
drexin bfffa33
WIP: More cases working
drexin 2d09a4b
WIP: Layout strings almost working
drexin 97b8ff6
Add layout string pointer to struct metadata
drexin c4b8bc4
Fetch bytecode layout strings from metadata in runtime
drexin fc81759
More efficient bytecode layout
drexin a80fc36
Add support for interpreted generics in layout strings
drexin 2636e38
Layout string instantiation, take and more
drexin 305df8b
Remove duplicate information from layout strings
drexin 02d4da0
Include size of previous object in next objects offset to reduce numb…
drexin c6d71c2
Add support for existentials
drexin 7b0d290
Build type layout strings with StructBuilder to support target sizes …
drexin 33d8c27
Add support for resilient types
drexin d73cc59
Properly cache layout strings in compiler
drexin 9e359ee
Generic resilient types working
drexin daa4d9f
Non-generic resilient types working
drexin c2cc170
Instantiate resilient type in layout when possible
drexin d846729
Fix a few issues around alignment and signing
drexin 5d88ea0
Disable generics, fix static alignment
drexin 8111b97
Fix MultiPayloadEnum size when no extra tag is necessary
drexin e5232e8
Fixes after rebase
drexin b889a30
Cleanup
drexin eaa25a1
Fix most tests
drexin 2081478
Fix objcImplementattion and non-Darwin builds
drexin 9bd88db
Fix BytecodeLayouts on non-Darwin
drexin 42202af
Fix Linux build
drexin 171e461
Fix sizes in linux tests
drexin 5a0ec36
Sign layout string pointers
drexin 3100ff0
Use nullptr instead of debug value
drexin 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
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
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
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
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
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.
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 whole file is outdated and should be removed/replaced with the current design