[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-22 Thread jakub at gcc dot gnu dot org


--- Comment #7 from jakub at gcc dot gnu dot org  2010-07-22 06:38 ---
Subject: Bug 45015

Author: jakub
Date: Thu Jul 22 06:38:25 2010
New Revision: 162397

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162397
Log:
PR debug/45015
* var-tracking.c (adjust_mems): Ignore ASM_OPERANDS with non-zero
ASM_OPERANDS_OUTPUT_IDX.
(adjust_insn): For inline asm with multiple sets ensure first
ASM_OPERANDS vectors are used by all following ASM_OPERANDS in
the insn.

* gcc.target/m68k/pr45015.c: New test.

Added:
branches/gcc-4_5-branch/gcc/testsuite/gcc.target/m68k/pr45015.c
Modified:
branches/gcc-4_5-branch/gcc/ChangeLog
branches/gcc-4_5-branch/gcc/testsuite/ChangeLog
branches/gcc-4_5-branch/gcc/var-tracking.c


-- 


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



[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-22 Thread jakub at gcc dot gnu dot org


--- Comment #8 from jakub at gcc dot gnu dot org  2010-07-22 06:46 ---
Should be fixed now.


-- 

jakub at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


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



[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-21 Thread jakub at gcc dot gnu dot org


-- 

jakub at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |jakub at gcc dot gnu dot org
   |dot org |
 Status|UNCONFIRMED |ASSIGNED
  Component|middle-end  |debug
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-07-21 12:29:16
   date||


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



[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-21 Thread jakub at gcc dot gnu dot org


--- Comment #2 from jakub at gcc dot gnu dot org  2010-07-21 12:40 ---
Created an attachment (id=21278)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21278action=view)
gcc46-pr45015.patch

Untested fix (together with testcase that fails even with current trunk without
the patch).  Will bootstrap/regtest on x86_64-linux and i686-linux, can you
please do the same on m68k?  Thanks.


-- 


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



[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-21 Thread mkuvyrkov at gcc dot gnu dot org


--- Comment #3 from mkuvyrkov at gcc dot gnu dot org  2010-07-21 17:58 
---
Created an attachment (id=21280)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21280action=view)
Testcase for patch

Thanks for looking into this problem!

The patch fixes the original testcase but causes a segmentation fault when
compiling libgcc.

.../cc1 -fpreprocessed libgcc2.i -quiet -dumpbase libgcc2.c -mcpu=54455
-auxbase-strip _muldi3.o -g -g -g -O2 -W -Wall -Wwrite-strings -Wcast-qual
-Wstrict-prototypes -Wmissing-prototypes -Wold-style-definition -version -fPIC
-fvisibility=hidden -fremove-local-statics -o libgcc2.s


-- 


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



[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-21 Thread jakub at gcc dot gnu dot org


--- Comment #4 from jakub at gcc dot gnu dot org  2010-07-21 18:05 ---
Can't reproduce that.  Were you testing the patch attached here, or the one
posted to gcc-patches?  The one attached here won't work when not
ENABLE_CHECKING...


-- 


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



[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-21 Thread maxim at codesourcery dot com


--- Comment #5 from mkuvyrkov at gcc dot gnu dot org  2010-07-21 18:20 
---
Subject: Re:  ICE in cselib.c caused by fix for PR43051

On 7/21/10 10:05 PM, jakub at gcc dot gnu dot org wrote:
 --- Comment #4 from jakub at gcc dot gnu dot org  2010-07-21 18:05 ---
 Can't reproduce that.  Were you testing the patch attached here, or the one
 posted to gcc-patches?  The one attached here won't work when not
 ENABLE_CHECKING...

That fixes the SIGSEGV.  Regtesting on m68k didn't show any regressions.

Thanks for fixing this.


-- 


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



[Bug debug/45015] ICE in cselib.c caused by fix for PR43051

2010-07-21 Thread jakub at gcc dot gnu dot org


--- Comment #6 from jakub at gcc dot gnu dot org  2010-07-21 19:12 ---
Subject: Bug 45015

Author: jakub
Date: Wed Jul 21 19:11:55 2010
New Revision: 162385

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=162385
Log:
PR debug/45015
* var-tracking.c (adjust_mems): Ignore ASM_OPERANDS with non-zero
ASM_OPERANDS_OUTPUT_IDX.
(adjust_insn): For inline asm with multiple sets ensure first
ASM_OPERANDS vectors are used by all following ASM_OPERANDS in
the insn.

* gcc.target/m68k/pr45015.c: New test.

Added:
trunk/gcc/testsuite/gcc.target/m68k/pr45015.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/var-tracking.c


-- 


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