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
