Skip to content
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

Fix HashMap/HashSet LLDB pretty-printer after hashbrown 0.11.0 #83920

Merged

Conversation

artemmukhin
Copy link
Contributor

@artemmukhin artemmukhin commented Apr 6, 2021

The pretty-printer was broken in #77566 after updating hashbrown to 0.11.0.
Note that the corresponding GDB pretty-printer was updated properly.

Fixes #83891

@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 6, 2021
@rust-log-analyzer

This comment has been minimized.

The pretty-printer was broken in rust-lang#77566
after updating hashbrown to 0.11.0.
Note that the corresponding GDB pretty-printer was updated properly.
@artemmukhin artemmukhin force-pushed the fix-hashmap-lldb-pretty-printer-1.52 branch from d057468 to 3d3a5ca Compare April 6, 2021 09:45
@artemmukhin artemmukhin changed the title Fix HashMap/HashSet LLDB pretty-printer on Rust 1.52 Fix HashMap/HashSet LLDB pretty-printer after hashbrown 0.11.0 Apr 6, 2021
artemmukhin added a commit to intellij-rust/intellij-rust that referenced this pull request Apr 6, 2021
@pnkfelix
Copy link
Member

pnkfelix commented Apr 6, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Apr 6, 2021

📌 Commit 3d3a5ca has been approved by pnkfelix

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 6, 2021
bors bot added a commit to intellij-rust/intellij-rust that referenced this pull request Apr 6, 2021
7059: Update HashMap/HashSet pretty-printers to Rust 1.52 r=Undin a=ortem

The corresponding PRs in rustc:
rust-lang/rust#77566
rust-lang/rust#83920

Fixes #7045

changelog: Update LLDB/GDB pretty-printers to render `HashMap`/`HashSet` on Rust 1.52 or higher

Co-authored-by: ortem <ortem00@gmail.com>
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 6, 2021
Rollup of 7 pull requests

Successful merges:

 - rust-lang#82963 (Move `SharedContext` to `context.rs`)
 - rust-lang#83829 (rustc_target: Rely on defaults more in target specs)
 - rust-lang#83895 (Add listing of lints (eg via `-W help`) to rustdoc)
 - rust-lang#83902 (Update LLVM to support more wasm simd ops)
 - rust-lang#83903 (Fix typo in TokenStream documentation)
 - rust-lang#83910 (Update cargo)
 - rust-lang#83920 (Fix HashMap/HashSet LLDB pretty-printer after hashbrown 0.11.0)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 3d33818 into rust-lang:master Apr 6, 2021
@rustbot rustbot added this to the 1.53.0 milestone Apr 6, 2021
Undin pushed a commit to intellij-rust/intellij-rust that referenced this pull request Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

lldb debuginfo test failure, hash_set/hash_map of size 0 instead of expected size 4?
7 participants