Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

171.swim fails on aarch64-linux as well. I dis a bisect and confirm it's r248877 causing the miscompare. Regards, Renlin On 06/06/17 12:05, Markus Trippelsdorf wrote: On 2017.06.05 at 22:39 +0200, Nicolas Koenig wrote: With all the style fixes committed as r248877. 171_swim fails now. I

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

On 2017.06.05 at 22:39 +0200, Nicolas Koenig wrote: > With all the style fixes committed as r248877. 171_swim fails now. I didn't bisect, but I suspect your revision. -- Markus

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

With all the style fixes committed as r248877. Thanks for the review. Nicolas On 06/03/2017 06:25 PM, Jerry DeLisle wrote: On 06/03/2017 06:48 AM, Nicolas Koenig wrote: Hello everyone, here is a version of the patch that includes a workaround for PR 80960. I have also included a separate

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

On Sat, Jun 03, 2017 at 09:25:31AM -0700, Jerry DeLisle wrote: > On 06/03/2017 06:48 AM, Nicolas Koenig wrote: > > Hello everyone, > > > > here is a version of the patch that includes a workaround for PR 80960. I > > have > > also included a separate test case for the failure that Dominique

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

On 06/03/2017 06:48 AM, Nicolas Koenig wrote: > Hello everyone, > > here is a version of the patch that includes a workaround for PR 80960. I have > also included a separate test case for the failure that Dominique detected. > The > style issues should be fixed. > > Regression-tested. OK for

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

Hello everyone, here is a version of the patch that includes a workaround for PR 80960. I have also included a separate test case for the failure that Dominique detected. The style issues should be fixed. Regression-tested. OK for trunk? Nicolas Changelog: 2017-06-03 Nicolas Koenig

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

> Le 1 juin 2017 à 16:19, Dominique d'Humières a écrit : > > I see > > FAIL: gfortran.dg/deferred_character_2.f90 -O1 execution test > FAIL: gfortran.dg/deferred_character_2.f90 -O2 execution test > FAIL: gfortran.dg/deferred_character_2.f90 -O3 -fomit-frame-pointer

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

> Le 1 juin 2017 à 11:30, Dominique d'Humières a écrit : > > >> Le 31 mai 2017 à 21:03, Nicolas Koenig a écrit : >> >> Hello Dominique, >> >> attached is the next try, this time without stupidities (I hope). Both test >> cases you posted don't

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

> Le 31 mai 2017 à 21:03, Nicolas Koenig a écrit : > > Hello Dominique, > > attached is the next try, this time without stupidities (I hope). Both test > cases you posted don't ICE anymore. > > Ok for trunk? > > Nicolas > Preliminary tests look OK, full testing

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

On 31 May 2017 at 21:03, Nicolas Koenig wrote: > Hello Dominique, > > attached is the next try, this time without stupidities (I hope). Both test > cases you posted don't ICE anymore. > > Ok for trunk? Please check contrib/check_GNU_style.sh /tmp/p8.diff and let me

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

Hello Dominique, attached is the next try, this time without stupidities (I hope). Both test cases you posted don't ICE anymore. Ok for trunk? Nicolas Regression tested for x86_64-pc-linux-gnu. Changelog (still the same): 2017-05-27 Nicolas Koenig PR

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

> Le 31 mai 2017 à 17:40, Dominique d'Humières a écrit : > > If I am not mistaken, compiling the following code with the patch applied simpler test print *,(huge(0),i=1,6) ! print*,(i,i=1,6) ! print*,(i,i=1,6,1) end > > gives an ICE. > > TIA > >

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

If I am not mistaken, compiling the following code with the patch applied program test_ivs use iso_varying_string implicit none type(varying_string),dimension(:,:),allocatable :: array2d type(varying_string) :: extra integer :: i,j allocate(array2d(2,3)) extra = "four"

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

On 29 May 2017 17:49:30 CEST, Nicolas Koenig wrote: >Hello Dominique, > >mea culpa, their was a bit confusion with the file being open in emacs >and vi at the same time. Attached is the new patch with the #define >removed. +static int +simplify_io_impl_do (gfc_code

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

> Le 29 mai 2017 à 17:49, Nicolas Koenig a écrit : > > Hello Dominique, > > mea culpa, their was a bit confusion with the file being open in emacs > and vi at the same time. Attached is the new patch with the #define removed. > > Nicolas > Thanks for the quick fix!

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

Hello Dominique, mea culpa, their was a bit confusion with the file being open in emacs and vi at the same time. Attached is the new patch with the #define removed. Nicolas On 05/29/2017 05:32 PM, Dominique d'Humières wrote: Hi Nicolas, Updating gfortran with your patch fails with

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

Hi Nicolas, Updating gfortran with your patch fails with ../../work/gcc/fortran/frontend-passes.c: In function 'bool traverse_io_block(gfc_code*, bool*, gfc_code*)': ../../work/gcc/fortran/frontend-passes.c:1067:20: error: expected unqualified-id before '(' token #define swap(x, y) (x) ^=

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

On 05/27/2017 12:49 PM, Nicolas Koenig wrote: Hello everyone, attached is a patch to simplify implied do loops in io statements by replacing them with their respective array slices. For example "WRITE (*,*) (a(i), i=1,4,2)" becomes "WRITE (*,*) a(1:4:2)". Ok for trunk? Thanks for patch.

[Patch, fortran] PR35339 Optimize implied do loops in io statements

Hello everyone, attached is a patch to simplify implied do loops in io statements by replacing them with their respective array slices. For example "WRITE (*,*) (a(i), i=1,4,2)" becomes "WRITE (*,*) a(1:4:2)". Ok for trunk? Nicolas Regression tested for x85_64-pc-linux-gnu. Changelog: