-
Notifications
You must be signed in to change notification settings - Fork 90
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
Implement #![debugger_visualizer] #1386
Comments
The function to insert this section is called at https://github.com/rust-lang/rust/blob/8c0b4f67c9b4bf477df38c16669fd576d46a2b3d/compiler/rustc_codegen_ssa/src/base.rs#L464 |
I'm interested in working on this issue. The code you've linked (https://github.com/rust-lang/rust/blob/ab0f3e680e4fd50218914b8afa6900a67377721c/compiler/rustc_codegen_llvm/src/debuginfo/gdb.rs) seems to already contain logic to add pretty-printers. What exactly is the issue here? |
That is the implementation in the LLVM backend. We need a separate implementation in rustc_codegen_cranelift. |
https://doc.rust-lang.org/nightly/reference/attributes/debugger.html#using-debugger_visualizer-with-gdb
https://github.com/rust-lang/rust/blob/ab0f3e680e4fd50218914b8afa6900a67377721c/compiler/rustc_codegen_llvm/src/debuginfo/gdb.rs
The
.debug_gdb_scripts
section should contain\x01gdb_load_rust_pretty_printers.py
followed by for each#![debugger_visualizer(gdb_script_file = "...")]
the pretty printer inlined into the executable. Except if one of the conditions to omit.debug_gdb_scripts
is true.The text was updated successfully, but these errors were encountered: