[PATCH] Fix PR debug/19327

Daniel Berlin dberlin@dberlin.org
Tue Jan 18 17:18:00 GMT 2005



On Tue, 18 Jan 2005, Eric Botcazou wrote:

>> Just FYI, it didn't introduce the regression.
>> That implies the patch is broken.
>
> Not in my mind.  AFAICT your patch is fine, but it wasn't tested with stabs,
> and caused a testcase that used to pass to fail; so we certainly regressed.

There are only a few platforms with stabs the default, and yes, i 
didn't test it on one of them :)

Note that this actually isn't going to make your optimized debug info much 
better anyway, because you are still going to be missing subblocks of 
blocks not marked used (see the PR that led up to the original patch for 
reasons why).
It just fixes the regression.
If you want to fix the whole problem, you'd also need to change 
dbxout_block to walk subblocks of blocks not marked used, instead of only 
walking subblocks of blocks marked used as it does now.

One of the reasons i didn't touch dbxout and do this when i did the same 
to dwarf2out is because i have no heavy stabs using platforms to test it 
on (darwin doesn't seem to break as easily)
--Dan



More information about the Gcc-patches mailing list