-
Notifications
You must be signed in to change notification settings - Fork 14k
[Debugger Visualizers] Unify *-gnu and *-msvc enum output
#147760
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
base: master
Are you sure you want to change the base?
Conversation
|
rustbot has assigned @Mark-Simulacrum. Use |
This comment has been minimized.
This comment has been minimized.
1d2d783 to
fe612ac
Compare
This comment has been minimized.
This comment has been minimized.
fe612ac to
d8b5ad9
Compare
|
This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
@bors r+ |
|
@bors rollup=iffy |
|
☔ The latest upstream changes (presumably #148721) made this pull request unmergeable. Please resolve the merge conflicts. |
Followup patch to #145218
Ports the msvc enum synthetic/summary logic to the gnu handlers so they have identical output. Also removes the "look-through-pointers" logic that caused #147450
I know we don't run the test suite in CI anymore, but it's still useful locally so I updated the relevant tests.