This is the mail archive of the gcc@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: recog.c apply_change_group ret zero if num_changes is 0.


> > * recog.c (apply_change_group): Return zero if no changes.
> 
> Don't do that.  That is not only in contradiction to the
> function's description, it'll break a lot of optimizations
> too, ones that test if a - possibly empty - set of required
> changes works before installing another change.

As an alternative it may make sense to make number of changes available
to the caller.  For instance GCSE pass did contain code that sometimes
makes no changes, but believe it did and causes re-iteration of
algorithm.

Honza
> 
> -- 
> --------------------------
> SuperH
> 2430 Aztec West / Almondsbury / BRISTOL / BS32 4AQ
> T:+44 1454 462330


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