Re: [PATCH] -gused

[Devang Patel <dpatel at apple dot com>]

On Thursday, June 19, 2003, at 10:09 PM, Richard Henderson wrote:

> On Thu, Jun 19, 2003 at 07:02:33PM -0700, Devang Patel wrote:
> > I found one bug in the patch I posted earlier. After fixing it gdb
> > dejagnu test reports 255 failures.
>
> Excellent.  This is exactly what I was hoping for.  ;-)
>
> So with drow's analysis of the additional failures that
> you didn't understand, I think the patch is probably on
> the right track.  Can you re-post it one more time?

Here it is. I'd like to continue to provide -gused for Darwin GCC 
users' benefit.
I am ready to add -feliminate-unused-debug-symbols flag in a separate
patch.

-Devang

2003-06-20  Devang Patel <dpatel@apple.com>

        * dbxout.c (dbxout_nesting): New static variable.
        (dbxout_flush_symbol_queue): New static function.
        (dbxout_queue_symbol): Same.
        (symbol_queue): New static variable.
        (symbol_queue_index): Same.
        (symbol_queue_size): Same.
        (DBXOUT_DECR_NESTING): New macro.
        (DBXOUT_DECR_NESTING_AND_RETURN): New macro.
        (dbxout_init): Delay symbol output.
        (dbxout_global_decl): Save, set and reset TREE_USED bit around 
dbxout_symbol() call.
        (dbxout_begin_function): Same.
        (dbxout_finish): Free symbol queue.
        (dbxout_type): Put appropriate symbols in queue.
        (dbxout_symbol): Put info for symbol's type in queue. 
Decrement/Increment nesting counts
        flush symbol queue appropriately.
        (dbxout_parms): Increment dbxout nesting.
        (dbxout_reg_parms): Same.
        * flags.h (flag_debug_only_used_symbols): New.
        * toplevl.c (flag_debug_only_used_symbols): New variable.
        (decode_g_option): Decode -gused.
        * doc/invoke.texi (Debugging Options): Document -gused.

Attachment: gused.take2.diff
Description: Binary data

(gused.take2.diff)