gprof cleanups

Ben Elliston
Fri Jul 9 04:26:00 GMT 2004

Ian Lance Taylor <> 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.


More information about the Binutils mailing list