-
Notifications
You must be signed in to change notification settings - Fork 527
Support separate debug symbol files (as in GDB) #253
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
Support separate debug symbol files (as in GDB) #253
Conversation
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.
I expect libgen.h header to exist when libdwarf is available, but that can be not true.
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.
This is a bit unrelated, but libelf installed on my PC (it's Arch Linux, but it's true also for Ubuntu 20.04 at least) does not have this definition, so the entire branch is omitted, so no 64-bit data, so no debuglink info. Probably we can work it around in a better way, but this line really stops me from having nice source locations.
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.
There are only 2 locations covered since I don't know what global debug directories contain.
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.
I expect debuglink to have only filename since, according to docs:
The executable contains a debug link that specifies the name of the separate debug info file.
Also, objcopy tool writes only the filename component in the .gnu_debuglink section even if an absolute path is provided:
https://github.com/bminor/binutils-gdb/blob/b4edb38e827974ecdbc4a6712d8b21bf876018a2/bfd/opncls.c#L1707-L1708
kobi-ca
commented
Sep 9, 2024
did this go anywhere?
current status for separate symbols file - need to be in the working dir?
thanks!
alexezeder
commented
Sep 13, 2024
did this go anywhere?
current status for separate symbols file - need to be in the working dir?
Based on what I can see, there is still no such support in master. Of course, this is not the greatest PR of mine (some things have to be done in CMake, and naming is not great either), but it looks like maintainers, aka @bombela, are not really interested in adding (削除) such (削除ここまで) functionality to this project. 🙂
I guess you can just use this branch for now because it provides the functionality you are looking for, or create your own fork for that. Anyway, I'm open to improving this PR to let it be merged into master.
Uh oh!
There was an error while loading. Please reload this page.
backward-cpp already supports loading debug symbols from a separate file. But currently, a separate file with debug symbols has to be located in the current working directory while the GDB documentation says:
This way, we have to look for a separate debug symbols file differently to replicate GDB behavior.
There are also a few points that I explained inline.