public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c
@ 2011-07-06 18:25 hjl.tools at gmail dot com
  2011-07-07  9:26 ` [Bug middle-end/49662] " rguenth at gcc dot gnu.org
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: hjl.tools at gmail dot com @ 2011-07-06 18:25 UTC (permalink / raw)
  To: gcc-bugs

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

           Summary: [4.7 Regression] FAIL:
                    gcc.dg/graphite/interchange-XX.c
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: hjl.tools@gmail.com
                CC: rguenth@gcc.gnu.org


On Linux/x86, revision 175916:

http://gcc.gnu.org/ml/gcc-cvs/2011-07/msg00181.html

caused:

FAIL: gcc.dg/graphite/interchange-14.c scan-tree-dump-times graphite "will be
interchanged" 1
FAIL: gcc.dg/graphite/interchange-15.c scan-tree-dump-times graphite "will be
interchanged" 1
FAIL: gcc.dg/graphite/interchange-mvt.c scan-tree-dump-times graphite "will be
interchanged" 1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug middle-end/49662] [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c
  2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
@ 2011-07-07  9:26 ` rguenth at gcc dot gnu.org
  2011-07-07 22:41 ` spop at gcc dot gnu.org
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-07-07  9:26 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.07.07 09:25:21
                 CC|                            |spop at gcc dot gnu.org
   Target Milestone|---                         |4.7.0
     Ever Confirmed|0                           |1

--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-07-07 09:25:21 UTC ---
PRE now confuses graphite.  No idea why and no chance I'll dive into graphite
to figure it out.  Maybe Sebastian can have a look, otherwise I'll XFAIL the
tests after a while (or add -fno-tree-pre to them).

My testing machine had too old ppl so I missed the fails.


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug middle-end/49662] [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c
  2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
  2011-07-07  9:26 ` [Bug middle-end/49662] " rguenth at gcc dot gnu.org
@ 2011-07-07 22:41 ` spop at gcc dot gnu.org
  2011-07-08  8:43 ` rguenth at gcc dot gnu.org
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: spop at gcc dot gnu.org @ 2011-07-07 22:41 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from Sebastian Pop <spop at gcc dot gnu.org> 2011-07-07 22:40:34 UTC ---
The loop nest is not perfect, in interchange-15.c:

  for (i = 0; i < N; i++)
    for (j = 0; j < N; j++)
      x[i] += a[j][i];

used to be perfect nest, now we have x[i] read and written in
the outer loop, enough to confuse loop interchange.

Please XFail the testcases, or change them to only have a
scalar reduction:

  for (i = 0; i < N; i++)
    for (j = 0; j < N; j++)
      x += a[j][i];


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug middle-end/49662] [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c
  2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
  2011-07-07  9:26 ` [Bug middle-end/49662] " rguenth at gcc dot gnu.org
  2011-07-07 22:41 ` spop at gcc dot gnu.org
@ 2011-07-08  8:43 ` rguenth at gcc dot gnu.org
  2011-07-08  8:44 ` [Bug tree-optimization/49662] [4.7 Regression] XFAIL: gcc.dg/graphite/interchange-XX.c, PRE confuses loop interchange rguenth at gcc dot gnu.org
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-07-08  8:43 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-07-08 08:42:36 UTC ---
Author: rguenth
Date: Fri Jul  8 08:42:31 2011
New Revision: 176031

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

    PR tree-optimization/49662
    * gcc.dg/graphite/interchange-14.c: XFAIL.
    * gcc.dg/graphite/interchange-15.c: Likewise.
    * gcc.dg/graphite/interchange-mvt.c: Likewise.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/graphite/interchange-14.c
    trunk/gcc/testsuite/gcc.dg/graphite/interchange-15.c
    trunk/gcc/testsuite/gcc.dg/graphite/interchange-mvt.c


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug tree-optimization/49662] [4.7 Regression] XFAIL: gcc.dg/graphite/interchange-XX.c, PRE confuses loop interchange
  2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
                   ` (2 preceding siblings ...)
  2011-07-08  8:43 ` rguenth at gcc dot gnu.org
@ 2011-07-08  8:44 ` rguenth at gcc dot gnu.org
  2011-09-30  9:41 ` ro at gcc dot gnu.org
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-07-08  8:44 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |missed-optimization, xfail
          Component|middle-end                  |tree-optimization
            Summary|[4.7 Regression] FAIL:      |[4.7 Regression] XFAIL:
                   |gcc.dg/graphite/interchange |gcc.dg/graphite/interchange
                   |-XX.c                       |-XX.c, PRE confuses loop
                   |                            |interchange

--- Comment #4 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-07-08 08:43:44 UTC ---
Now XFAILed.


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug tree-optimization/49662] [4.7 Regression] XFAIL: gcc.dg/graphite/interchange-XX.c, PRE confuses loop interchange
  2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
                   ` (3 preceding siblings ...)
  2011-07-08  8:44 ` [Bug tree-optimization/49662] [4.7 Regression] XFAIL: gcc.dg/graphite/interchange-XX.c, PRE confuses loop interchange rguenth at gcc dot gnu.org
@ 2011-09-30  9:41 ` ro at gcc dot gnu.org
  2011-09-30  9:59 ` rguenther at suse dot de
  2011-10-10 11:53 ` rguenth at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: ro at gcc dot gnu.org @ 2011-09-30  9:41 UTC (permalink / raw)
  To: gcc-bugs

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

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ro at gcc dot gnu.org

--- Comment #5 from Rainer Orth <ro at gcc dot gnu.org> 2011-09-30 09:36:48 UTC ---
All 3 tests now XPASS everywhere.  It seems the XFAILs can be removed?

  Rainer


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug tree-optimization/49662] [4.7 Regression] XFAIL: gcc.dg/graphite/interchange-XX.c, PRE confuses loop interchange
  2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
                   ` (4 preceding siblings ...)
  2011-09-30  9:41 ` ro at gcc dot gnu.org
@ 2011-09-30  9:59 ` rguenther at suse dot de
  2011-10-10 11:53 ` rguenth at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: rguenther at suse dot de @ 2011-09-30  9:59 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #6 from rguenther at suse dot de <rguenther at suse dot de> 2011-09-30 09:53:26 UTC ---
On Fri, 30 Sep 2011, ro at gcc dot gnu.org wrote:

> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49662
> 
> Rainer Orth <ro at gcc dot gnu.org> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                  CC|                            |ro at gcc dot gnu.org
> 
> --- Comment #5 from Rainer Orth <ro at gcc dot gnu.org> 2011-09-30 09:36:48 UTC ---
> All 3 tests now XPASS everywhere.  It seems the XFAILs can be removed?

Looks like so


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug tree-optimization/49662] [4.7 Regression] XFAIL: gcc.dg/graphite/interchange-XX.c, PRE confuses loop interchange
  2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
                   ` (5 preceding siblings ...)
  2011-09-30  9:59 ` rguenther at suse dot de
@ 2011-10-10 11:53 ` rguenth at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-10-10 11:53 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #7 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-10-10 11:53:02 UTC ---
Fixed.


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2011-10-10 11:53 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-06 18:25 [Bug middle-end/49662] New: [4.7 Regression] FAIL: gcc.dg/graphite/interchange-XX.c hjl.tools at gmail dot com
2011-07-07  9:26 ` [Bug middle-end/49662] " rguenth at gcc dot gnu.org
2011-07-07 22:41 ` spop at gcc dot gnu.org
2011-07-08  8:43 ` rguenth at gcc dot gnu.org
2011-07-08  8:44 ` [Bug tree-optimization/49662] [4.7 Regression] XFAIL: gcc.dg/graphite/interchange-XX.c, PRE confuses loop interchange rguenth at gcc dot gnu.org
2011-09-30  9:41 ` ro at gcc dot gnu.org
2011-09-30  9:59 ` rguenther at suse dot de
2011-10-10 11:53 ` rguenth at gcc dot gnu.org

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).