* [PATCH, testsuite]: Fix scan string of gfortran.dg/intrinsic_size_3.f90
@ 2013-01-09 19:15 Uros Bizjak
2013-01-09 19:29 ` Tobias Burnus
0 siblings, 1 reply; 2+ messages in thread
From: Uros Bizjak @ 2013-01-09 19:15 UTC (permalink / raw)
To: gcc-patches; +Cc: Fortran List
[-- Attachment #1: Type: text/plain, Size: 479 bytes --]
Hello!
The regexp in the scan string assumes that dumped temporaries have
exactly 4 numbers. Sometimes, this is not the case.
Attached patch fixes scan string to match a number with arbitrary
number of places instead.
2012-01-09 Uros Bizjak <ubizjak@gmail.com>
* gfortran.dg/intrinsic_size_3.f90: Make scan-tree-dump-times
number matching more robust.
Tested on x86_64-pc-linux-gnu and alphaev68-linux-gnu, where it fixes
spurious test failure.
OK for mainline?
Uros.
[-- Attachment #2: t.diff.txt --]
[-- Type: text/plain, Size: 641 bytes --]
Index: gfortran.dg/intrinsic_size_3.f90
===================================================================
--- gfortran.dg/intrinsic_size_3.f90 (revision 195057)
+++ gfortran.dg/intrinsic_size_3.f90 (working copy)
@@ -22,5 +22,5 @@
stop
end program bug
-! { dg-final { scan-tree-dump-times "iszs = \\(integer\\(kind=2\\)\\) MAX_EXPR <\\(D.....->dim.0..ubound - D.....->dim.0..lbound\\) \\+ 1, 0>;" 1 "original" } }
+! { dg-final { scan-tree-dump-times "iszs = \\(integer\\(kind=2\\)\\) MAX_EXPR <\\(D.\[0-9\]+->dim.0..ubound - D.\[0-9\]+->dim.0..lbound\\) \\+ 1, 0>;" 1 "original" } }
! { dg-final { cleanup-tree-dump "original" } }
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH, testsuite]: Fix scan string of gfortran.dg/intrinsic_size_3.f90
2013-01-09 19:15 [PATCH, testsuite]: Fix scan string of gfortran.dg/intrinsic_size_3.f90 Uros Bizjak
@ 2013-01-09 19:29 ` Tobias Burnus
0 siblings, 0 replies; 2+ messages in thread
From: Tobias Burnus @ 2013-01-09 19:29 UTC (permalink / raw)
To: Uros Bizjak; +Cc: gcc-patches, Fortran List
The patch is OK. Thanks!
Tobias
Uros Bizjak wrote:
> The regexp in the scan string assumes that dumped temporaries have
> exactly 4 numbers. Sometimes, this is not the case.
>
> Attached patch fixes scan string to match a number with arbitrary
> number of places instead.
>
> 2012-01-09 Uros Bizjak <ubizjak@gmail.com>
>
> * gfortran.dg/intrinsic_size_3.f90: Make scan-tree-dump-times
> number matching more robust.
>
> Tested on x86_64-pc-linux-gnu and alphaev68-linux-gnu, where it fixes
> spurious test failure.
>
> OK for mainline?
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-01-09 19:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-09 19:15 [PATCH, testsuite]: Fix scan string of gfortran.dg/intrinsic_size_3.f90 Uros Bizjak
2013-01-09 19:29 ` Tobias Burnus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).