[Patch, testsuite] PR61453 gfortran.dg/bind_c_array_params_2.f90 for targets where a call insn isn't "call"
John David Anglin
dave.anglin@bell.net
Sun Jul 6 14:42:00 GMT 2014
I would change hppa*-*-hpux* to hppa*-*-*. That's what I tested on
hppa-linux.
I guess if "\[ \t\]_*myBindC" worked on hppa*-*-linux*, then it would
probably work
on hpux and the target condition could be removed. The reason hpux
needed
special treatment originally is for every function call to an external
symbol, there
is an .import directive containing the symbol name.
Dave
On 6-Jul-14, at 5:22 AM, Dominique Dhumieres wrote:
>> On hppa, the ",%r2" uniquely identifies the call. The test passes
>> with -flto.
>
> Then is the following patch OK for hppa*-*-*?
>
> +++ gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-07-05
> 22:20:49.000000000 +0200
> @@ -16,7 +16,7 @@ integer :: aa(4,4)
> call test(aa)
> end
>
> -! { dg-final { scan-assembler-times "call\[^\n\r\]*myBindC" 1
> { target { ! { hppa*-*-hpux* } } } } }
> -! { dg-final { scan-assembler-times "call\[^\n\r\]*myBindC,%r2" 1
> { target { hppa*-*-hpux* } } } }
> +! { dg-final { scan-assembler-times "\[ \t\]_*myBindC" 1 { target
> { ! { hppa*-*-hpux* } } } } }
> +! { dg-final { scan-assembler-times "myBindC,%r2" 1 { target
> { hppa*-*-hpux* } } } }
> ! { dg-final { scan-tree-dump-times "test \\\(&parm\\." 1
> "original" } }
> ! { dg-final { cleanup-tree-dump "original" } }
>
> Dominique
>
--
John David Anglin dave.anglin@bell.net
More information about the Fortran
mailing list