Re: [PATCH] Kill TYPE_METHODS debug 1/9

[Nathan Sidwell <nathan at acm dot org>]

On 07/18/2017 01:24 PM, Jim Wilson wrote:

> Changes to the debug info files requires a gdb make check with and 
> without the patch to check for regressions.  Since you are changing both 
> dbxout and dwarf2out, you would need to do this twice, once for each 
> debug info type.  Testing dbxout may be a little tricky, since few 
> systems still use this by default.  Maybe you can hack an x86_64-linux 
> build to use dbxout by default to test it.

DWARF results are unchanged.

DBX results after hacking x86-64 to prefer stabs are too horrible to 
tell.  However, a manual check on a simple program shows that gdb 
couldn't locate member functions anyway even before the patch and an 
unhacked g++.

g++ -gstabs:
(gdb) ptype X
type = struct X {
    int m;
}

g++ -gdwarf:
(gdb) ptype X
type = struct X {
    int m;
  public:
    int frob(int);
    X(int);
}

So I don't think I've made it worse there.   thoughts?

nathan

--
Nathan Sidwell