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

2010-06-14 Thread 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=gccview=revrev=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

2010-06-14 Thread 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



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

2010-06-11 Thread 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

2010-06-11 Thread 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=20892action=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

2010-06-11 Thread 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