Skip to content

[lldb][enums] Remove broadcast bits from debugger (#91618) #8729

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

Conversation

chelcassanova
Copy link

Removes the debugger broadcast bits from Debugger.h and instead uses the enum from lldb-enumerations.h and adds the
eBroadcastSymbolChange bit to the enum in lldb-enumerations.h. This fixes a bug wherein the incorrect broadcast bit could be referenced due both of these enums previously existing and being out-of-sync with each other.

(cherry picked from commit b9e3fa8)

Removes the debugger broadcast bits from `Debugger.h` and instead uses
the enum from `lldb-enumerations.h` and adds the
`eBroadcastSymbolChange` bit to the enum in `lldb-enumerations.h`. This fixes a bug wherein the incorrect broadcast bit could be referenced due both of these enums previously existing and being out-of-sync with each other.

(cherry picked from commit b9e3fa8)
@chelcassanova chelcassanova requested a review from JDevlieghere May 9, 2024 18:17
@chelcassanova
Copy link
Author

@swift-ci please test

Copy link

@JDevlieghere JDevlieghere left a comment

Choose a reason for hiding this comment

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

Hmm, why is swig generating new enum values? I wouldn't have expected the bindings to change as you're changing the private enum. I guess the bindings must have been out of date? Regardless, the result is correct.

LGTM

@JDevlieghere JDevlieghere merged commit 5345565 into swiftlang:swift/release/6.0 May 9, 2024
3 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.

2 participants