-
-
Notifications
You must be signed in to change notification settings - Fork 610
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
Emit standard DWARF3 opcode for TLS address #11087
Conversation
DW_OP_GNU_push_tls_address is a GNU extension prior to DWARF2. DWARF3 introduced DW_OP_form_tls_address with the exact same semantic, as can be seen here: - https://sourceware.org/legacy-ml/gdb-patches/2016-08/msg00222.html - https://reviews.llvm.org/rL274366
|
Thanks for your pull request, @Geod24! Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. Testing this PR locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub run digger -- build "master + dmd#11087" |
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.
Looks fine to me.
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.
Ok. Compiler should support the latest gdb/dwarf features, and gdb should support old versions of compilers. Not the other way around.
|
Sigh |
|
Saved it for the future. |
DW_OP_GNU_push_tls_address is a GNU extension prior to DWARF2.
DWARF3 introduced DW_OP_form_tls_address with the exact same semantic, as can be seen here:
Caveat: GDB support was "only" added in 7.12, which was released on 2016-10-07.
Since I don't use DMD (debug infos are utterly broken on MacOSX), I didn't observe any downside of this, but I was going over the DWARF code and found this oddity and figured it was time to replace it.
CC @ibuclaw