public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "dblaikie at gmail dot com" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug debug/49312] Make DW_AT_name contain only simple name, no template-id Date: Thu, 12 Jan 2023 18:29:36 +0000 [thread overview] Message-ID: <bug-49312-4-bU7yS31p1Q@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-49312-4@http.gcc.gnu.org/bugzilla/> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49312 David Blaikie <dblaikie at gmail dot com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dblaikie at gmail dot com --- Comment #4 from David Blaikie <dblaikie at gmail dot com> --- FWIW I've (partially) implemented this in Clang under the flag `-gsimple-template-names` due to some really large debug info issues with (especially eigen and tensorflow) expression templates (I saw template names as long as 50k characters leading to exceeding the 32 bit limit in .debug_str.dwo sections in dwp files (and gold dwp didn't check for overflow, so this caused silent corruption)). There certainly are some issues with it - my approach didn't simplify all names - certain names aren't easy to roundtrip from the DIE descriptions of the template parameters (lambdas are a great/difficult example, for instance - the lambda type DIEs don't have anything about the lambda mangling number, etc, and maybe should - so even in non-template cases the lambdas could be matched up between two TUs (lambdas in inline functions are the same type even in different translation units & should be treated as such)) So there's a bunch of work that'd probably need to be done on template DIE accuracy/completeness before this feature could be adopted wholesale without any exemptions, but for the size benefits (especially in expression template heavy code) I've found it to be worthwhile & we've done some work to flesh out support for this in lldb as well as identify maybe a couple of gdb bugs related to this. I've tried to poke dwarf-discuss about the lambda issue in particular ( http://lists.dwarfstd.org/pipermail/dwarf-discuss-dwarfstd.org/2022-August/007117.html ) but no one seems interested in discussing it. So maybe it's something GCC/Clang/GDB/LLDB folks should discuss more directly together. Also some recent discussion with Simon Marchi on the gdb list: https://sourceware.org/pipermail/gdb/2023-January/050496.html
prev parent reply other threads:[~2023-01-12 18:29 UTC|newest] Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-06-07 16:49 [Bug debug/49312] New: " dodji at gcc dot gnu.org 2011-06-07 16:50 ` [Bug debug/49312] " dodji at gcc dot gnu.org 2011-06-07 22:11 ` dodji at gcc dot gnu.org 2014-04-29 16:26 ` tromey at gcc dot gnu.org 2023-01-12 18:29 ` dblaikie at gmail dot com [this message]
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=bug-49312-4-bU7yS31p1Q@http.gcc.gnu.org/bugzilla/ \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).