[Patch, fortran] Improve common function elimination
Tobias Burnus
burnus@net-b.de
Mon Jan 2 11:34:00 GMT 2012
Thomas Koenig wrote:
> --- dependency.c (Revision 182430)
> +++ dependency.c (Arbeitskopie)
> @@ -245,7 +245,9 @@ gfc_dep_compare_functions (gfc_expr *e1, gfc_expr
> * 0 if e1 == e2
> * -1 if e1< e2
> * -2 if the relationship could not be determined
> - * -3 if e1 /= e2, but we cannot tell which one is larger. */
> + * -3 if e1 /= e2, but we cannot tell which one is larger.
> + REAL and COMPLEX constants are only compared for equality
> + or inequality; if they are unequal, -2 is returned in all cases. */
Can we finally move to an ENUM? I think we slowly drift into the realm
of magic numbers ...
(I know that you didn't modify that part.)
Steve Kargl wrote:
> On Wed, Dec 28, 2011 at 04:21:55PM +0100, Thomas Koenig wrote:
>>> OK for trunk?
> I did not test the patch, but it appears correct to me.
> OK.
Same here: Not tested, but it looks OK.
Tobias
More information about the Gcc-patches
mailing list