* [Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c
2010-05-11 13:11 [Bug middle-end/44078] New: [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c hjl dot tools at gmail dot com
@ 2010-05-11 13:26 ` rguenth at gcc dot gnu dot org
2010-05-11 13:43 ` borntraeger at de dot ibm dot com
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2010-05-11 13:26 UTC (permalink / raw)
To: gcc-bugs
--
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c
2010-05-11 13:11 [Bug middle-end/44078] New: [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c hjl dot tools at gmail dot com
2010-05-11 13:26 ` [Bug middle-end/44078] " rguenth at gcc dot gnu dot org
@ 2010-05-11 13:43 ` borntraeger at de dot ibm dot com
2010-05-11 13:57 ` borntraeger at de dot ibm dot com
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: borntraeger at de dot ibm dot com @ 2010-05-11 13:43 UTC (permalink / raw)
To: gcc-bugs
------- 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c
2010-05-11 13:11 [Bug middle-end/44078] New: [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c hjl dot tools at gmail dot com
2010-05-11 13:26 ` [Bug middle-end/44078] " rguenth at gcc dot gnu dot org
2010-05-11 13:43 ` borntraeger at de dot ibm dot com
@ 2010-05-11 13:57 ` borntraeger at de dot ibm dot com
2010-05-15 1:31 ` hjl dot tools at gmail dot com
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: borntraeger at de dot ibm dot com @ 2010-05-11 13:57 UTC (permalink / raw)
To: gcc-bugs
------- 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c
2010-05-11 13:11 [Bug middle-end/44078] New: [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c hjl dot tools at gmail dot com
` (2 preceding siblings ...)
2010-05-11 13:57 ` borntraeger at de dot ibm dot com
@ 2010-05-15 1:31 ` hjl dot tools at gmail dot com
2010-05-17 7:54 ` krebbel at gcc dot gnu dot org
2010-07-23 11:52 ` rguenth at gcc dot gnu dot org
5 siblings, 0 replies; 7+ messages in thread
From: hjl dot tools at gmail dot com @ 2010-05-15 1:31 UTC (permalink / raw)
To: gcc-bugs
------- 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c
2010-05-11 13:11 [Bug middle-end/44078] New: [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c hjl dot tools at gmail dot com
` (3 preceding siblings ...)
2010-05-15 1:31 ` hjl dot tools at gmail dot com
@ 2010-05-17 7:54 ` krebbel at gcc dot gnu dot org
2010-07-23 11:52 ` rguenth at gcc dot gnu dot org
5 siblings, 0 replies; 7+ messages in thread
From: krebbel at gcc dot gnu dot org @ 2010-05-17 7:54 UTC (permalink / raw)
To: gcc-bugs
------- 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug middle-end/44078] [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c
2010-05-11 13:11 [Bug middle-end/44078] New: [4.6 regression] FAIL: gcc.dg/tree-ssa/prefetch-7.c hjl dot tools at gmail dot com
` (4 preceding siblings ...)
2010-05-17 7:54 ` krebbel at gcc dot gnu dot org
@ 2010-07-23 11:52 ` rguenth at gcc dot gnu dot org
5 siblings, 0 replies; 7+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2010-07-23 11:52 UTC (permalink / raw)
To: gcc-bugs
------- 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
^ permalink raw reply [flat|nested] 7+ messages in thread