Skip to content

[DebugInfo] Stop emitting spare bits mask in debug info #77038

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
merged 3 commits into from
Oct 17, 2024

Conversation

augusto2112
Copy link
Contributor

We're now able to calculate the spare bits mask from other information. Stop emitting it in debug info.

We're now able to calculate the spare bits mask from other information.
Stop emitting it in debug info.
@augusto2112
Copy link
Contributor Author

@augusto2112
Copy link
Contributor Author

without relying on spare bit information in the reflection metadata
(which was added in swiftlang#40906).  As a result, we can remove the
code from swiftlang#40906.

This is the first step in such removal.  It removes the RemoteMirror
code for looking up such metadata.  It leaves behind:

* Sufficient stubs for LLDB to continue to build.  Once LLDB is updated, these stubs can be removed as well.

* The compiler code to emit such metadata.  This allows new binaries to still reflect MPEs on older runtimes.  This will need to be kept for a transitional period.

(cherry picked from commit c20ef6d)
@augusto2112
Copy link
Contributor Author

@augusto2112 augusto2112 requested a review from tbkka October 16, 2024 19:11
Copy link
Contributor

@adrian-prantl adrian-prantl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice if we don't need it any more!

@adrian-prantl
Copy link
Contributor

I'll defer to @tbkka to accept this.

Remote Mirrors is able to calculate type infos without needing
MultiPayloadEnumDescriptors. Remove the functionality to look it up.
@augusto2112 augusto2112 force-pushed the stop-spare-bits-debug branch from f628bac to 6b704ad Compare October 16, 2024 20:21
@augusto2112
Copy link
Contributor Author

@augusto2112 augusto2112 merged commit 17926e7 into swiftlang:rebranch Oct 17, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants