Bug 45266 - [4.6 regression] FAIL: gfortran.dg/array_memcpy_3.f90
Summary: [4.6 regression] FAIL: gfortran.dg/array_memcpy_3.f90
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: testsuite (show other bugs)
Version: 4.6.0
: P3 normal
Target Milestone: 4.6.0
Assignee: Not yet assigned to anyone
URL:
Keywords:
: 45275 (view as bug list)
Depends on:
Blocks: 45545
  Show dependency treegraph
 
Reported: 2010-08-12 15:43 UTC by H.J. Lu
Modified: 2010-08-17 10:47 UTC (History)
4 users (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed: 2010-08-12 15:50:34


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description H.J. Lu 2010-08-12 15:43:04 UTC
On Linux/x86-64, revision 163191 gave

FAIL: gfortran.dg/array_memcpy_3.f90  -O  scan-tree-dump-times original "memcpy|(ref-all.*ref-all)" 2
FAIL: gfortran.dg/array_memcpy_3.f90  -O  scan-tree-dump-times original "memcpy|(ref-all.*ref-all)" 2

Revision 163187 is OK. This may be caused by revision 163189:

http://gcc.gnu.org/ml/gcc-cvs/2010-08/msg00400.html

Linux/ia32 doesn't have this regression.
Comment 1 Richard Biener 2010-08-12 15:46:55 UTC
The pattern doesn't match even though I see two memcpy calls!?
Comment 2 H.J. Lu 2010-08-12 15:48:56 UTC
(In reply to comment #1)
> The pattern doesn't match even though I see two memcpy calls!?
> 

I am using

# make RUNTESTFLAGS="--target_board 'unix{-m32,}'" check

2 failures are 1 for 64bit and 1 for 32bit.
Comment 3 Richard Biener 2010-08-12 15:50:34 UTC
I see the fails, but the pattern should still match.
Comment 4 H.J. Lu 2010-08-12 16:44:04 UTC
I was wrong. Linux/ia32 has the same regression:

FAIL: gfortran.dg/array_memcpy_3.f90  -O  scan-tree-dump-times original "memcpy|(ref-all.*ref-all)" 2
Comment 5 Richard Biener 2010-08-12 18:16:47 UTC
Pls don't change back bug attributes all the time...
Comment 6 Richard Biener 2010-08-13 12:35:23 UTC
*** Bug 45275 has been marked as a duplicate of this bug. ***
Comment 7 Steve Ellcey 2010-08-13 22:39:00 UTC
Does "memcpy|(ref-all.*ref-all)"  need to be "(memcpy|(ref-all.*ref-all))" or
perhaps "(memcpy|ref-all.*ref-all)".  Everyplace else I see a | in a scan
statement there are parentheses around the options.
Comment 8 Steve Ellcey 2010-08-16 17:11:46 UTC
! { dg-final { scan-tree-dump-times "memcpy|ref-all.*ref-all" 2 "original" } }

worked for me on IA64 where we have 2 memcpys' in the output.  Neither of my
suggestions from comment #7 worked.  I don't know if this will work on a platform
which generates the 'ref-all' strings instead of memcpy's.  All my platforms; IA64, PA, x86 seem to generate memcpy.
Comment 9 Richard Biener 2010-08-17 09:47:57 UTC
Subject: Bug 45266

Author: rguenth
Date: Tue Aug 17 09:47:44 2010
New Revision: 163297

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=163297
Log:
2010-08-17  Richard Guenther  <rguenther@suse.de>

	PR testsuite/45266
	* gfortran.dg/array_memcpy_3.f90: Adjust pattern.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gfortran.dg/array_memcpy_3.f90

Comment 10 Jakub Jelinek 2010-08-17 10:47:30 UTC
Fixed then.