gprof cleanups

Ben Elliston bje@au1.ibm.com
Fri Jul 9 04:26:00 GMT 2004


Ian Lance Taylor <ian@wasabisystems.com> writes:

> I don't understand the point of this loop either before or after your
> patch.  Doesn't it just duplicate work down by the loop just a few
> lines up?
> 
>   for (sp = &default_excluded_list[0]; *sp; sp++)
>     {
>       sym_id_add (*sp, EXCL_TIME);
>       sym_id_add (*sp, EXCL_GRAPH);
>       sym_id_add (*sp, EXCL_FLAT);
>     }

I finally got around to looking at this and I agree with you.  I'll
submit a proper patch next week that removes the duplicated code.

Ben



More information about the Binutils mailing list