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

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

[hjl dot tools at gmail dot com]

On Linux/ia32, revision 159262 gave:

FAIL: gcc.dg/tree-ssa/prefetch-7.c scan-tree-dump-times aprefetch "nontemporal
store" 2

Revision 159255 is OK. It may be caused by revisions 159256/159257:

http://gcc.gnu.org/ml/gcc-cvs/2010-05/msg00307.html
http://gcc.gnu.org/ml/gcc-cvs/2010-05/msg00308.html


-- 
           Summary: [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hjl dot tools at gmail dot com


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=20629&action=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

[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=20629&action=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

[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=gcc&view=rev&rev=159475
Log:
2010-05-17  Christian Borntraeger  <borntraeger@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

[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