-
Notifications
You must be signed in to change notification settings - Fork 13.6k
[libc++] Update GDB pretty-printer to work with GDB 17 #142106
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: main
Are you sure you want to change the base?
Conversation
Fixed an issue in `libcxx/utils/gdb/libcxx/printers.py`. With gdb 17 (binutils 2_44) pretty-printers do not work anymore because calls to `gdb.printing` require `import gdb.printing` statement, which was missing from the printers.py Broken after commit bminor/binutils-gdb@fc14343, and `importgdb.printing` was first referenced in bminor/binutils-gdb@ee06c79
Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it is probably because you do not have write permissions for the repository. In which case you can instead tag reviewers by name in a comment by using If you have received no comments on your PR for a week, you can request a review by "ping"ing the PR by adding a comment “Ping”. The common courtesy "ping" rate is once a week. Please remember that you are asking for valuable time from other developers. If you have further questions, they may be answered by the LLVM GitHub User Guide. You can also ask questions in a comment on this PR, on the LLVM Discord or on the forums. |
@llvm/pr-subscribers-libcxx Author: kotborealis (kotborealis) ChangesFixed an issue in With gdb 17 (binutils 2_44) pretty-printers do not work anymore because calls to Broken after commit binutils-gdb/fc14343205d3a68db1fc139e4af9796be208fab4, and Full diff: https://github.com/llvm/llvm-project/pull/142106.diff 1 Files Affected:
diff --git a/libcxx/utils/gdb/libcxx/printers.py b/libcxx/utils/gdb/libcxx/printers.py
index e3d5d87aca325..90bc54d987ee8 100644
--- a/libcxx/utils/gdb/libcxx/printers.py
+++ b/libcxx/utils/gdb/libcxx/printers.py
@@ -14,6 +14,7 @@
import re
import gdb
+import gdb.printing
# One under-documented feature of the gdb pretty-printer API
# is that clients can call any other member of the API
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix. LGTM, pending CI.
The branch was updated to pass CI which had infrastructural errors |
Yeah, we're having a lot of infra issues. If it fails again, don't do anything, I'll restart just the jobs that failed. |
Fixed an issue in
libcxx/utils/gdb/libcxx/printers.py
.With gdb 17 (binutils 2_44) pretty-printers do not work anymore because calls to
gdb.printing
requiresimport gdb.printing
statement, which was missing from theprinters.py
.Broken after commit binutils-gdb/fc14343205d3a68db1fc139e4af9796be208fab4, and
import gdb.printing
was first referenced in binutils-gdb/ee06c79b0fefd5e4ed5e7a1171dc3440130e41da