This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran 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: {patch, fortran] PR38718: add simplifier for MERGE

From: Daniel Franke <franke dot daniel at gmail dot com>
To: Jerry DeLisle <jvdelisle at verizon dot net>
Cc: Daniel Kraft <d at domob dot eu>, gcc-patches at gcc dot gnu dot org, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>
Date: Sun, 4 Jan 2009 17:10:54 +0100
Subject: Re: {patch, fortran] PR38718: add simplifier for MERGE
References: <200901041534.31699.franke.daniel@gmail.com> <4960D000.4030104@domob.eu> <4960DC01.2030001@verizon.net>

On Sunday 04 January 2009 16:55:45 Jerry DeLisle wrote:
> >> Regression tested on i686-pc-linux-gnu. Ok for trunk?
> >
> > Ok.  I'm not sure whether we should wait for 4.5, but my opinion is that
> > the change is quite clear and it should be ok for trunk, it actually
> > fixes a rejects-valid bug.
> >
> > Thanks for fixing this!
>
> I concur, a rejects valid is OK.

Comitted as r143053.

Also, while at it, I also committed r143054. There I removed the last TODO 
item from intrinsic.c (do simplify), line 3338 . The check in question is 
already implemented in intrinsic.c (gfc_intrinsic_func_interface), around line 
3700.

Cheers

	Daniel

References:
- {patch, fortran] PR38718: add simplifier for MERGE
  - From: Daniel Franke
- Re: {patch, fortran] PR38718: add simplifier for MERGE
  - From: Daniel Kraft
- Re: {patch, fortran] PR38718: add simplifier for MERGE
  - From: Jerry DeLisle

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