Right commit revision, wrong attached file (original patch, not the follow-up one).
Now hopefully the correct one.

Tobias

Am 15.10.18 um 21:02 schrieb Tobias Burnus:
Fixed with commit Rev. 265175 as attached.

Cheers

Tobias


Dominique d'Humières wrote:

Le 14 oct. 2018 à 00:43, Tobias Burnus <bur...@net-b.de> a écrit :


Dominique d'Humières wrote:
UNRESOLVED: gfortran.dg/inline_matmul_24.f90   -O0 scan-tree-dump-times optimized "gamma5[__var_1_do \\\\* 4 \\\\+ __var_2_do]" 1

! { dg-final { scan-tree-dump-times "gamma5\[__var_1_do \\* 4 \\+ __var_2_do\]" 1 "optimized" } }

Shouldn’t -fdump-tree-original be -fdump-tree-optimized?
As it is a front-end optimization (which is explicitly enabled), -fdump-tree-original should work (and avoids issues with further later optimizations).

What do you get on your system? Seemingly something else than I do. Can you look for the gamma5 line in your dump?

Tobias
Then

! { dg-final { scan-tree-dump-times "gamma5\[__var_1_do \\* 4 \\+ __var_2_do\]" 1 "optimized" } }

should be

! { dg-final { scan-tree-dump-times "gamma5\[__var_1_do \\* 4 \\+ __var_2_do\]" 1 "original" } }

isn’t it?

see https://gcc.gnu.org/ml/gcc-testresults/2018-10/msg01721.html for non darwin log.

Dominique
Index: gcc/testsuite/gfortran.dg/inline_matmul_24.f90
===================================================================
--- gcc/testsuite/gfortran.dg/inline_matmul_24.f90	(Revision 265174)
+++ gcc/testsuite/gfortran.dg/inline_matmul_24.f90	(Revision 265175)
@@ -39,4 +39,4 @@
       call abort()
     end if
 end program testMATMUL
-! { dg-final { scan-tree-dump-times "gamma5\[__var_1_do \\* 4 \\+ __var_2_do\]" 1 "optimized" } }
+! { dg-final { scan-tree-dump-times "gamma5\\\[__var_1_do \\* 4 \\+ __var_2_do\\\]|gamma5\\\[NON_LVALUE_EXPR <__var_1_do> \\* 4 \\+ NON_LVALUE_EXPR <__var_2_do>\\\]" 1 "original" } }
Index: gcc/testsuite/ChangeLog
===================================================================
--- gcc/testsuite/ChangeLog	(Revision 265174)
+++ gcc/testsuite/ChangeLog	(Revision 265175)
@@ -1,3 +1,8 @@
+2018-10-15  Tobias Burnus  <bur...@net-b.de>
+
+	PR fortran/87597
+	* gfortran.dg/inline_matmul_24.f90: Tweak scan-tree.
+
 2018-10-15  Renlin Li  <renlin...@arm.com>
 
 	PR target/87563

Reply via email to