[Bug middle-end/43656] New: "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[Bug middle-end/43656] New: "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[zsojka at seznam dot cz]

Command line:
gcc -O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity
-fcompare-debug testcase.c

Tested revisions:
r157965 - fail
r157877 - fail
r153685 - fail


-- 
           Summary: "-fcompare-debug failure" with "-O2 -fschedule-insns -
                    fsched-pressure -funroll-loops -fgraphite-identity"
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: zsojka at seznam dot cz
  GCC host triplet: x86_64-pc-linux-gnu
GCC target triplet: x86_64-pc-linux-gnu


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

[Bug middle-end/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[zsojka at seznam dot cz]

------- Comment #1 from zsojka at seznam dot cz  2010-04-05 22:31 -------
Created an attachment (id=20318)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20318&action=view)
reduced testcase

Command line:
gcc -O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity
-fcompare-debug pr43656.c


-- 


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

[Bug middle-end/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[jakub at gcc dot gnu dot org]

------- Comment #2 from jakub at gcc dot gnu dot org  2010-04-08 11:53 -------
Can't reproduce this with r158104.


-- 


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

[Bug middle-end/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[zsojka at seznam dot cz]

------- Comment #3 from zsojka at seznam dot cz  2010-04-08 12:21 -------
Created an attachment (id=20334)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20334&action=view)
testcase that crashes in r158095

When size of arrays is changed from 64 to 1000, it fails again in r158095


-- 

zsojka at seznam dot cz changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #20318|0                           |1
        is obsolete|                            |


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

[Bug middle-end/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[zsojka at seznam dot cz]

------- Comment #4 from zsojka at seznam dot cz  2010-05-22 18:00 -------
This, nor the original (unreduced) testcase, no longer fails in r159387,
r159511 and r159696.


-- 


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

[Bug debug/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[aoliva at gcc dot gnu dot org]

------- Comment #5 from aoliva at gcc dot gnu dot org  2010-06-11 13:57 -------
Mine


-- 

aoliva at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |aoliva at gcc dot gnu dot
                   |dot org                     |org


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

[Bug debug/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[aoliva at gcc dot gnu dot org]

------- Comment #6 from aoliva at gcc dot gnu dot org  2010-06-11 14:00 -------
Created an attachment (id=20892)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20892&action=view)
Patch that fixes the problem

This patch (perhaps in addition to the patch for PR 44181, I haven't tried the
old tree without that patch) fixes sched to disregard any apparent register
pressure from debug insns.  I'm going to give it some more testing before
posting it.


-- 


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

[Bug debug/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[zsojka at seznam dot cz]

------- Comment #7 from zsojka at seznam dot cz  2010-06-11 16:10 -------
I can confirm the testcase from comment #3 doesn't fail in r158095 when only
patch from comment #6 is applied (and gcc_checking_assert() is replaced by
gcc_assert())


-- 


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

[Bug debug/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[aoliva at gcc dot gnu dot org]

------- Comment #8 from aoliva at gcc dot gnu dot org  2010-06-14 19:18 -------
Subject: Bug 43656

Author: aoliva
Date: Mon Jun 14 19:18:04 2010
New Revision: 160761

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160761
Log:
PR debug/43656
* haifa-sched.c (setup_insn_reg_pressure_info,
update_register_pressure): Reject debug insns.
(ready_sort): Don't setup reg pressure for debug insns.
(schedule_insn): Don't update reg pressure for debug insns.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/haifa-sched.c


-- 


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

[Bug debug/43656] "-fcompare-debug failure" with "-O2 -fschedule-insns -fsched-pressure -funroll-loops -fgraphite-identity"

[aoliva at gcc dot gnu dot org]

------- Comment #9 from aoliva at gcc dot gnu dot org  2010-06-14 19:19 -------
Fixed


-- 

aoliva at gcc dot gnu dot org changed:

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


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