diff mbox

Fix regex for assembly test on AIX PowerPC

Message ID 20170320173041.777e36ef@gunter.codethink.co.uk
State New
Headers show

Commit Message

Violeta Menendez Gonzalez March 20, 2017, 5:30 p.m. UTC 
Hi,

I've tested this with GCC 7.0 on powerpc-ibm-aix7.2.0.0

This test adds periods as a valid character for symbol names. Periods
are allowed in symbol names but I am unsure if they have a special
meaning at the beginning of a name that changes the meaning of this
test. Currently my GFortran is generating `bl .myBindC` (branch with
link) and I can't find any reasons why this would be wrong, but I also
can't find a reason why the GFortran is adding this period here.
Testing in x86_64 I get `call myBindC`.

2017-03-20 Violeta Menendez Gonzalez <violeta.menendez@codethink.co.uk>

	* gfortran.dg/bind_c_array_params_2.f90: Allow symbol names
	to contain periods.
diff mbox

Patch

diff --git a/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 b/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90
index e749958..3418893 100644
--- a/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90
+++ b/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90
@@ -16,7 +16,7 @@  integer :: aa(4,4)
 call test(aa)
 end
 
-! { dg-final { scan-assembler-times "\[ \t\]\[$,_0-9\]*myBindC" 1 { target { ! { hppa*-*-* s390*-*-* } } } } }
+! { dg-final { scan-assembler-times "\[ \t\]\[$.,_0-9\]*myBindC" 1 { target { ! { hppa*-*-* s390*-*-* } } } } }
 ! { dg-final { scan-assembler-times "myBindC,%r2" 1 { target { hppa*-*-* } } } }
 ! { dg-final { scan-assembler-times "brasl\t%r\[0-9\]*,myBindC" 1 { target { s390*-*-* } } } }
 ! { dg-final { scan-tree-dump-times "test \\\(&parm\\." 1 "original" } }