recog.c apply_change_group ret zero if num_changes is 0.

recog.c apply_change_group ret zero if num_changes is 0.

[Graham Stott]

All,

I noticed that if apply_change_group is called and num_changes is zero
then 1 is returned. I think it makes more sense to return 0 if no
changes are outstanding.

Bootstrapped i686-pc-linux-gnu all languages including ada no regressions.

ChangeLog

	* recog.c (apply_change_group): Return zero if no changes.

Graham

-------------------------------------------------------------------
Index: gcc/recog.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/recog.c,v
retrieving revision 1.160
diff -c -p -r1.160 recog.c
*** gcc/recog.c 28 Jun 2002 15:43:53 -0000      1.160
--- gcc/recog.c 13 Jul 2002 12:19:25 -0000
*************** apply_change_group ()
*** 317,322 ****
--- 317,326 ----
     int i;
     rtx last_validated = NULL_RTX;

+   /* If no changes return zero.  */
+   if (num_changes == 0)
+     return 0;
+
     /* The changes have been applied and all INSN_CODEs have been reset to force
        rerecognition.

----------------------------------------------------------------------

Re: recog.c apply_change_group ret zero if num_changes is 0.

[Richard Henderson]

On Sat, Jul 13, 2002 at 07:45:18PM +0100, Graham Stott wrote:
> I noticed that if apply_change_group is called and num_changes is zero
> then 1 is returned. I think it makes more sense to return 0 if no
> changes are outstanding.

Possibly, possibly not.  It is true that all substitutions
were successful, which might be important to some pass that
may or may not remember whether or not it made changes while
determining if some (larger) change is possible.

What is this supposed to affect?


r~