[Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c

[rguenth at gcc dot gnu dot org]

--- Comment #5 from rguenth at gcc dot gnu dot org  2010-07-23 11:52 ---
Fixed.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44078

[Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c

[krebbel at gcc dot gnu dot org]

--- Comment #4 from krebbel at gcc dot gnu dot org  2010-05-17 07:54 ---
Subject: Bug 44078

Author: krebbel
Date: Mon May 17 07:53:20 2010
New Revision: 159475

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=159475
Log:
2010-05-17  Christian Borntraeger  borntrae...@de.ibm.com

PR 44078
* gcc.dg/tree-ssa/prefetch-7.c: Change pattern to match only the old
debug messages but not the newly introduced one.



Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/tree-ssa/prefetch-7.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44078

[Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c

[hjl dot tools at gmail dot com]

--- Comment #3 from hjl dot tools at gmail dot com  2010-05-15 01:31 ---
(In reply to comment #2)
 Created an attachment (id=20629)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20629action=view) [edit]
 Testfix for the prefetch-7.c testcase
 
 There always was
 fprintf (dump_file, Marked reference %p as a nontemporal store.\n,
 in the old code.
 
 r159256 now adds
 fprintf (dump_file, Ignoring nontemporal store %p\n, (void *) ref);
 
 The testcase matches both messages, but it should only match the first one.
 
 So a potential fix might look like like the attachement. Can you confirm that
 this patch fixes the testcase on your system?
 

This fixed the problem. Thanks.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44078

[Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

   Target Milestone|--- |4.6.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44078

[Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c

[borntraeger at de dot ibm dot com]

--- Comment #1 from borntraeger at de dot ibm dot com  2010-05-11 13:43 
---
From a first look this looks like that the test case scans for
nontemporal store which is also emitted by the new debug messages:

-return false;
+{
+  if (dump_file  (dump_flags  TDF_DETAILS))
+fprintf (dump_file, Ignoring nontemporal store %p\n, (void *) ref);
+  return false;
+}

I will check if that is the problem. If yes I will probably fix the testcase.

Christian


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44078

[Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c

[borntraeger at de dot ibm dot com]

--- Comment #2 from borntraeger at de dot ibm dot com  2010-05-11 13:57 
---
Created an attachment (id=20629)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20629action=view)
Testfix for the prefetch-7.c testcase

There always was
fprintf (dump_file, Marked reference %p as a nontemporal store.\n,
in the old code.

r159256 now adds
fprintf (dump_file, Ignoring nontemporal store %p\n, (void *) ref);

The testcase matches both messages, but it should only match the first one.

So a potential fix might look like like the attachement. Can you confirm that
this patch fixes the testcase on your system?

Christian


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44078