Re: *ping* - Re: [Patch, Fortran] Add end-of-scope finalization (Part 2 of 2)

2013-07-09 Thread Dominique Dhumieres
Dear Tobias, The following patch prevents the warning to generate a FAIL: --- /opt/gcc/_clean/gcc/testsuite/gfortran.dg/class_48.f90 2012-02-05 22:03:27.0 +0100 +++ /opt/gcc/work/gcc/testsuite/gfortran.dg/class_48.f902013-07-01 19:44:06.0 +0200 @@ -158,4 +158,6 @@ c

Re: *ping* - Re: [Patch, Fortran] Add end-of-scope finalization (Part 2 of 2)

2013-06-28 Thread Dominique Dhumieres
Dear Tobias, Nit pick: I find that '(\\(integer\\(kind=8\\)\\) )?' is easier to parse than '(\\(integer\\(kind=8\\)\\) |)' (a matter of taste and I won't argue about it). > > /opt/gcc/work/gcc/testsuite/gfortran.dg/class_48.f90: In function > > '__final_test2_T.2138.constprop.0': > > /opt/gcc/w

*ping* - Re: [Patch, Fortran] Add end-of-scope finalization (Part 2 of 2)

2013-06-28 Thread Tobias Burnus
Ping. Change: Updated scan-tree-dump for -m32 / -m64 differences. Tobias Burnus wrote: This patch adds finalization calls for components. This completes the end-of-scope finalization, but it is also called for the LHS of intrinsic assignment. (LHS finalization for the variable itself is still

Re: [Patch, Fortran] Add end-of-scope finalization (Part 2 of 2)

2013-06-26 Thread Dominique Dhumieres
Dear Tobias, Failures in 32 bit mode again!-( With your patch, I get the following failures with -m32: FAIL: gfortran.dg/class_48.f90 -O2 (test for excess errors) FAIL: gfortran.dg/class_48.f90 -Os (test for excess errors) FAIL: gfortran.dg/finalize_18.f90 -O scan-tree-dump-times original

[Patch, Fortran] Add end-of-scope finalization (Part 2 of 2)

2013-06-25 Thread Tobias Burnus
This patch adds finalization calls for components. This completes the end-of-scope finalization, but it is also called for the LHS of intrinsic assignment. (LHS finalization for the variable itself is still lacking.) Build and regtested on x86-64-gnu-linux. OK for the trunk? Tobias 2013-06-25