http://d.puremagic.com/issues/show_bug.cgi?id=4149


Robert Clipsham <robert@octarineparrot.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch


--- Comment #9 from Robert Clipsham <robert@octarineparrot.com> 2010-05-09 15:29:05 BST ---
If this patch is accepted, we should probably change the DW_AT_language to be C++ instead of C with -gc, and update the help message to say so.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=4149



--- Comment #10 from Walter Bright <bugzilla@digitalmars.com> 2012-01-18 20:52:37 PST ---
Can this be turned into a pull request?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=4149



--- Comment #11 from Brad Roberts <braddr@puremagic.com> 2012-01-18 20:55:55 PST ---
Last time I played with this diff on current code, it caused failures.  I haven't looked at in any detail in ages and I'm fairly sure it's rather incomplete now that I know more about the backend.

The bug is still real though.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=4149


Leandro Lucarella <leandro.lucarella@sociomantic.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |leandro.lucarella@sociomant
                   |                            |ic.com


--- Comment #12 from Leandro Lucarella <leandro.lucarella@sociomantic.com> 2012-01-26 07:04:10 PST ---
(In reply to comment #6)
> I agree with Brad's comment 2.

What's the relation between this and pull request 526[1] (commit e81dbc2 in dmd-1.x) that has been merged recently. At least according to the pull request, the improved usage of DWARF is only activated when -g is specified. Should that be changed for -gc? What about bug 3389? I think Robert's suggestion about having a -gd for D extensions and -g for standard debugging stuff would be better.

Right now the situation with -g/-gc is far from ideal.

[1] https://github.com/D-Programming-Language/dmd/pull/526

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=4149


dawg@dawgfoto.de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dawg@dawgfoto.de
         AssignedTo|nobody@puremagic.com        |dawg@dawgfoto.de


--- Comment #13 from dawg@dawgfoto.de 2012-01-31 13:52:52 PST ---
It's unrelated to pull 526 which fixed #4180.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=4149



--- Comment #14 from Leandro Lucarella <leandro.lucarella@sociomantic.com> 2012-02-01 02:22:04 PST ---
(In reply to comment #13)
> It's unrelated to pull 526 which fixed #4180.

I'm talking about the comment 2 by Brad:

> Depends on if gc implies c, c++, or 'as much as is supported in the built-in debug format without extension'.
> 
> I'd argue for the last definition.  Given that dwarf supports it without extension, I'd argue that it should use it.  I'd argue that -g should be built-in + d extensions.

Walter expressed he agree with this comment in comment 3.

AFAIK this pull request works for -g instead of -gc, even when it uses standard DWARF features, which goes against that comment.

In the bug 3389 that you just closed, Robert Clipsham makes what for me is a good point:

> -g not working is how it's meant to be, at least until gdb adds support for the D extensions to DWARF. I've hopefully fixed the remaining bugs with -gc (on linux at least). My solution to this would be to add in a -gd, and make -g an alias to -gc until better support for debug info is added to debuggers. This way users get working debug output with -g, and don't blame it on a buggy dmd :)

At least this will be more familiar with people used to GCC command line arguments (which is probably 100% of the *nix world), where -g is the default for debug and you have, for example -ggdb for GDB extensions.

AFAIK there is no clear direction about this. But maybe is a topic for another bug report, I think I'll reopen bug 3389 because the root issue in that bug is what's being discussed here.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=4149



--- Comment #15 from Brad Roberts <braddr@puremagic.com> 2012-02-01 21:03:43 PST ---
Please stop cluttering this bug report with discussion about the meaning of -g, that's a solved problem.  This report is about preserving 'ref'ness through the backend and dwarf debug data.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=4149



--- Comment #16 from dawg@dawgfoto.de 2012-02-02 15:44:42 PST ---
https://github.com/D-Programming-Language/dmd/pull/687

Based on patch but had to add another change to e2ir.c.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------