On Mon, Jan 14, 2013 at 10:51:25PM +0100, Thomas Koenig wrote:
> Am 14.01.2013 14:29, schrieb Mikael Morin:
> > Le 13/01/2013 23:14, Thomas Koenig a écrit :
>
> > > OK for trunk?
> > >
> > OK with the changes suggested above. Thanks.
>
> Committed as rev. 195179 with your changes.
s/intrnisics/
Am 14.01.2013 14:29, schrieb Mikael Morin:
Le 13/01/2013 23:14, Thomas Koenig a écrit :
OK for trunk?
OK with the changes suggested above. Thanks.
Committed as rev. 195179 with your changes.
Thanks a lot for the thorough review!
Thomas
Le 13/01/2013 23:14, Thomas Koenig a écrit :
Hi Mikael,
thanks a lot for your comments!
+ actual_arglist->expr = gfc_copy_expr (e); +
actual_arglist->next = gfc_get_actual_arglist ();
Another one is needed. I get a segmentation fault with SUM.
Fixed by using gfc_build_intrisic_call.
Nice.
Hi Mikael,
thanks a lot for your comments!
+ actual_arglist->expr = gfc_copy_expr (e);
+ actual_arglist->next = gfc_get_actual_arglist ();
Another one is needed. I get a segmentation fault with SUM.
Fixed by using gfc_build_intrisic_call. I have also put SUM
into the test case.
Le 01/01/2013 21:18, Thomas Koenig a écrit :
Hello world,
the attached patch replaces ANY(a, b, c) with a .or. b .or c,
leading to reduced execution time. It also handles ALL, PRODUCT
and SUM.
This fixes a bug noted by Michael Metcalf.
Regression-tested. OK for trunk?
A few comments below.
Ping**2?
This was submitted before the review, so I think it should still be OK.
Ping?
http://gcc.gnu.org/ml/fortran/2013-01/msg0.html
Hello world,
the attached patch replaces ANY(a, b, c) with a .or. b .or c,
leading to reduced execution time. It also handles ALL, PRODUCT
and SUM.
T
Ping?
http://gcc.gnu.org/ml/fortran/2013-01/msg0.html
Hello world,
the attached patch replaces ANY(a, b, c) with a .or. b .or c,
leading to reduced execution time. It also handles ALL, PRODUCT
and SUM.
This fixes a bug noted by Michael Metcalf.
Regression-tested. OK for trunk?
Hello world,
the attached patch replaces ANY(a, b, c) with a .or. b .or c,
leading to reduced execution time. It also handles ALL, PRODUCT
and SUM.
This fixes a bug noted by Michael Metcalf.
Regression-tested. OK for trunk?
Thomas
2013-01-01 Thomas Koenig
PR fortran/5580
