This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Two redundant calls of bitmap_bit_p

>> +	* tree-into-ssa.c (mark_block_for_update): Avoid redundant call
>> +	of bitmap_bit_p.
>> +	* cfganal.c (compute_dominance_frontiers_1): Likewise.
>>    
> This is fine.  I suspect there's quite a few places that do similar stuff.

For sure, I would probably write such code myself having a chance :)
I learnt about set_bit return value only recently.  Those two appeared in my
oprofile runs, but it might make sense to review others.

Honza
>
> Jeff
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]