public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/37053]  New: ICE in reload_cse_simplify_operands, at postreload.c:395
@ 2008-08-08  1:02 stephen at marenka dot net
  2008-08-08  1:03 ` [Bug c/37053] " stephen at marenka dot net
                   ` (21 more replies)
  0 siblings, 22 replies; 31+ messages in thread
From: stephen at marenka dot net @ 2008-08-08  1:02 UTC (permalink / raw)
  To: gcc-bugs

gcc -fpic -O1 -c postreload.c

Dropping optimization from -O1 to -O0 or dropping -fpic allows this to succeed.

Using built-in specs.
Target: m68k-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Debian 4.3.1-8'
--with-bugurl=file:///usr/share/doc/gcc-4.3/README.Bugs
--enable-languages=c,c++,fortran,objc,obj-c++ --prefix=/usr --enable-shared
--with-system-zlib --libexecdir=/usr/lib --without-included-gettext
--enable-threads=posix --enable-nls --with-gxx-include-dir=/usr/include/c++/4.3
--program-suffix=-4.3 --enable-clocale=gnu --enable-libstdcxx-debug
--enable-objc-gc --enable-mpfr --disable-libssp --disable-werror
--enable-checking=release --build=m68k-linux-gnu --host=m68k-linux-gnu
--target=m68k-linux-gnu
Thread model: posix
gcc version 4.3.1 (Debian 4.3.1-8)
COLLECT_GCC_OPTIONS='-save-temps' '-v' '-fpic' '-O1' '-c' '-m68020'
 /usr/lib/gcc/m68k-linux-gnu/4.3.1/cc1 -E -quiet -v postreload.c -m68020 -fpic
-O1 -fpch-preprocess -o postreload.i
ignoring nonexistent directory "/usr/local/include/m68k-linux-gnu"
ignoring nonexistent directory
"/usr/lib/gcc/m68k-linux-gnu/4.3.1/../../../../m68k-linux-gnu/include"
ignoring nonexistent directory "/usr/include/m68k-linux-gnu"
#include "..." search starts here:
#include <...> search starts here:
 /usr/local/include
 /usr/lib/gcc/m68k-linux-gnu/4.3.1/include
 /usr/lib/gcc/m68k-linux-gnu/4.3.1/include-fixed
 /usr/include
End of search list.
COLLECT_GCC_OPTIONS='-save-temps' '-v' '-fpic' '-O1' '-c' '-m68020'
 /usr/lib/gcc/m68k-linux-gnu/4.3.1/cc1 -fpreprocessed postreload.i -quiet
-dumpbase postreload.c -m68020 -auxbase postreload -O1 -version -fpic -o
postreload.s
GNU C (Debian 4.3.1-8) version 4.3.1 (m68k-linux-gnu)
        compiled by GNU C version 4.3.1, GMP version 4.2.2, MPFR version 2.3.1.
GGC heuristics: --param ggc-min-expand=82 --param ggc-min-heapsize=98603
Compiler executable checksum: 941413abde0b2d7fe947ae106f766200
postreload.c: In function 'do_termsform':
postreload.c:745: warning: assignment makes pointer from integer without a cast
postreload.c:806: warning: initialization makes pointer from integer without a
cast
postreload.c:842: warning: passing argument 1 of 'Rf_install' makes pointer
from integer without a cast
postreload.c:845: warning: passing argument 1 of 'Rf_install' makes pointer
from integer without a cast
postreload.c:845: warning: assignment makes pointer from integer without a cast
postreload.c: At top level:
postreload.c:488: warning: 'EncodeVars' used but never defined
postreload.c:533: warning: 'ExpandDots' used but never defined
postreload.c: In function 'do_termsform':
postreload.c:886: error: insn does not satisfy its constraints:
(insn 309 2675 2677 36 postreload.c:446 (set (reg:SI 0 %d0)
        (plus:SI (mem/f:SI (reg:SI 8 %a0) [0 S4 A16])
            (reg:SI 0 %d0))) 132 {*addsi3_internal} (nil))
postreload.c:886: internal compiler error: in reload_cse_simplify_operands, at
postreload.c:395
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.3/README.Bugs> for instructions.


-- 
           Summary: ICE in reload_cse_simplify_operands, at postreload.c:395
           Product: gcc
           Version: 4.3.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: stephen at marenka dot net
 GCC build triplet: m68k-linux-gnu
  GCC host triplet: m68k-linux-gnu
GCC target triplet: m68k-linux-gnu


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


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

* [Bug c/37053] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
@ 2008-08-08  1:03 ` stephen at marenka dot net
  2008-08-08  1:04 ` stephen at marenka dot net
                   ` (20 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: stephen at marenka dot net @ 2008-08-08  1:03 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from stephen at marenka dot net  2008-08-08 01:02 -------
Created an attachment (id=16040)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16040&action=view)
Preprocessed file


-- 


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


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

* [Bug c/37053] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
  2008-08-08  1:03 ` [Bug c/37053] " stephen at marenka dot net
@ 2008-08-08  1:04 ` stephen at marenka dot net
  2008-09-11 15:21 ` [Bug middle-end/37053] [4.3/4.4 regression] " schwab at suse dot de
                   ` (19 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: stephen at marenka dot net @ 2008-08-08  1:04 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from stephen at marenka dot net  2008-08-08 01:02 -------
Created an attachment (id=16041)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16041&action=view)
reduced source from r-base


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
  2008-08-08  1:03 ` [Bug c/37053] " stephen at marenka dot net
  2008-08-08  1:04 ` stephen at marenka dot net
@ 2008-09-11 15:21 ` schwab at suse dot de
  2008-09-11 15:30 ` bonzini at gnu dot org
                   ` (18 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: schwab at suse dot de @ 2008-09-11 15:21 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from schwab at suse dot de  2008-09-11 15:19 -------
Caused by:

2007-07-23  Peter Bergner  <bergner@vnet.ibm.com>
            Jakub Jelinek  <jakub@redhat.com>

        PR middle-end/PR28690

Can be reproduced with gcc.c-torture/execute/20060420-1.c when compiled with
-O2.


-- 

schwab at suse dot de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bergner at vnet dot ibm dot
                   |                            |com, jakub at redhat dot com
OtherBugsDependingO|                            |28690
              nThis|                            |
             Status|UNCONFIRMED                 |NEW
          Component|target                      |middle-end
     Ever Confirmed|0                           |1
  GCC build triplet|m68k-linux-gnu              |
   GCC host triplet|m68k-linux-gnu              |
           Keywords|                            |ice-on-valid-code
   Last reconfirmed|0000-00-00 00:00:00         |2008-09-11 15:19:57
               date|                            |
            Summary|ICE in                      |[4.3/4.4 regression] ICE in
                   |reload_cse_simplify_operands|reload_cse_simplify_operands
                   |, at postreload.c:395       |, at postreload.c:395
   Target Milestone|---                         |4.3.4


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (2 preceding siblings ...)
  2008-09-11 15:21 ` [Bug middle-end/37053] [4.3/4.4 regression] " schwab at suse dot de
@ 2008-09-11 15:30 ` bonzini at gnu dot org
  2008-09-16 21:01 ` schwab at suse dot de
                   ` (17 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2008-09-11 15:30 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #4 from bonzini at gnu dot org  2008-09-11 15:29 -------
I think there is a missing constrain_operands somewhere, because in

(define_insn "*addsi3_internal"
  [(set (match_operand:SI 0 "nonimmediate_operand" "=m,?a,?a,d,a")
        (plus:SI (match_operand:SI 1 "general_operand" "%0,a,rJK,0,0")
                 (match_operand:SI 2 "general_src_operand"
"dIKLT,rJK,a,mSrIKLT,mSrIKLs")))]

the insn does match the fourth alternative with operands 1 and 2 commuted.


-- 

bonzini at gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bonzini at gnu dot org


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (3 preceding siblings ...)
  2008-09-11 15:30 ` bonzini at gnu dot org
@ 2008-09-16 21:01 ` schwab at suse dot de
  2008-09-16 21:03 ` bonzini at gnu dot org
                   ` (16 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: schwab at suse dot de @ 2008-09-16 21:01 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #5 from schwab at suse dot de  2008-09-16 21:00 -------
Reverting this change in commutative_operand_precedence fixes the testcase:

     case RTX_OBJ:
       /* Complex expressions should be the first, so decrease priority
-         of objects.  */
-      return -1;
+         of objects.  Prefer pointer objects over non pointer objects.  */
+      if ((REG_P (op) && REG_POINTER (op))
+         || (MEM_P (op) && MEM_POINTER (op)))
+       return -1;
+      return -2;

     case RTX_COMM_ARITH:
       /* Prefer operands that are themselves commutative to be first.


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (4 preceding siblings ...)
  2008-09-16 21:01 ` schwab at suse dot de
@ 2008-09-16 21:03 ` bonzini at gnu dot org
  2008-09-20 14:30 ` rguenth at gcc dot gnu dot org
                   ` (15 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2008-09-16 21:03 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #6 from bonzini at gnu dot org  2008-09-16 21:02 -------
But that was the meat of fixing PR28690. :-(

The insn should satisfy the constraints of alternative 4.


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (5 preceding siblings ...)
  2008-09-16 21:03 ` bonzini at gnu dot org
@ 2008-09-20 14:30 ` rguenth at gcc dot gnu dot org
  2009-06-23 11:40 ` [Bug middle-end/37053] [4.3/4.4/4.5 " mkuvyrkov at gcc dot gnu dot org
                   ` (14 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2008-09-20 14:30 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #7 from rguenth at gcc dot gnu dot org  2008-09-20 14:29 -------
m68k-linux-gnu is neither primary nor secondary target.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (6 preceding siblings ...)
  2008-09-20 14:30 ` rguenth at gcc dot gnu dot org
@ 2009-06-23 11:40 ` mkuvyrkov at gcc dot gnu dot org
  2009-06-23 12:05 ` bonzini at gnu dot org
                   ` (13 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: mkuvyrkov at gcc dot gnu dot org @ 2009-06-23 11:40 UTC (permalink / raw)
  To: gcc-bugs

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1747 bytes --]



------- Comment #8 from mkuvyrkov at gcc dot gnu dot org  2009-06-23 11:39 -------
(In reply to comment #6)
> But that was the meat of fixing PR28690. :-(
> 
> The insn should satisfy the constraints of alternative 4.

Well, not really.  For the insn to match alternative 4 the pattern should be
non-canonical.  The last sentence in GCC Internals' canonicalization rules
says:

"Further canonicalization rules are defined in the function
commutative_operand_ 
precedence in ‘gcc/rtlanal.c’"

Unfortunately, commutative_operand_precendence() at the moment clearly states
that a pointer (being that a MEM or a REG) has precedence over other RTX_OBJs. 
It is absolutely unclear to me why a pointer should have precedence over, say,
multiplication or anything else (and "yes", I've read PR28690).

With my target-independent hat on, I would remove that PPC-specific hunk from
commutative_operand_precendence() or, if that is really that important to PPC,
add a new target hook so that different targets can enjoy privilege of defining
that to whatever they seem fit.  Adding such an obscure canonicalization rule
for all targets seems unjustified.

I'd like to get some feedback on the above before I start implementing new
target hook to make all targets happy.

Peter, I'm CCing you as the author of the commutative_operand_precendence()
piece  to get your opinion on the above.


-- 

mkuvyrkov at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mkuvyrkov at gcc dot gnu dot
                   |                            |org


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (7 preceding siblings ...)
  2009-06-23 11:40 ` [Bug middle-end/37053] [4.3/4.4/4.5 " mkuvyrkov at gcc dot gnu dot org
@ 2009-06-23 12:05 ` bonzini at gnu dot org
  2009-06-23 12:26 ` mkuvyrkov at gcc dot gnu dot org
                   ` (12 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2009-06-23 12:05 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #9 from bonzini at gnu dot org  2009-06-23 12:04 -------
But % makes it commutative, no?  So operand 2 matches operand 0, and operand 1
matches mSrIKLT.


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (8 preceding siblings ...)
  2009-06-23 12:05 ` bonzini at gnu dot org
@ 2009-06-23 12:26 ` mkuvyrkov at gcc dot gnu dot org
  2009-06-23 13:09 ` bonzini at gnu dot org
                   ` (11 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: mkuvyrkov at gcc dot gnu dot org @ 2009-06-23 12:26 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #10 from mkuvyrkov at gcc dot gnu dot org  2009-06-23 12:26 -------
(In reply to comment #9)
> But % makes it commutative, no?

Yes, but that only means that the operands can be swapped *if*
swap_commutative_operands_p() returns true.  Due to the funny precedence that
does not happen.

> So operand 2 matches operand 0, and operand 1
> matches mSrIKLT.

Matching procedures do not take commutativeness into account.

Part of the problem are optimizations using commutativeness during reload.  One
way to paper over the issue is to forbid such optimizations during reload, but
that may worsen code.  And this approach is not as clean as letting backends
decide if their .md files can handle funny canonicalization rules.


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (9 preceding siblings ...)
  2009-06-23 12:26 ` mkuvyrkov at gcc dot gnu dot org
@ 2009-06-23 13:09 ` bonzini at gnu dot org
  2009-06-23 17:22 ` mkuvyrkov at gcc dot gnu dot org
                   ` (10 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2009-06-23 13:09 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #11 from bonzini at gnu dot org  2009-06-23 13:09 -------
> Yes, but that only means that the operands can be swapped *if*
> swap_commutative_operands_p() returns true.  Due to the funny precedence that
> does not happen.
>
> Matching procedures do not take commutativeness into account.

That would mean that PR28690 screwed up two-address machines big.  In that case
my opinion of the patch would change.

Still, I don't think a target hook is the solution.  Even if it adds hack over
hack, having the "funny precedence" rules only before reload could be a
solution.

I'm pretty sure that we are talking about two different things. :-)


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (10 preceding siblings ...)
  2009-06-23 13:09 ` bonzini at gnu dot org
@ 2009-06-23 17:22 ` mkuvyrkov at gcc dot gnu dot org
  2009-06-24 16:02 ` mkuvyrkov at gcc dot gnu dot org
                   ` (9 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: mkuvyrkov at gcc dot gnu dot org @ 2009-06-23 17:22 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #12 from mkuvyrkov at gcc dot gnu dot org  2009-06-23 17:21 -------
(In reply to comment #11)
> Still, I don't think a target hook is the solution.  Even if it adds hack over
> hack, having the "funny precedence" rules only before reload could be a
> solution.

For the record, I consider the hook to be a feature, the one which backends can
use to tweak code generation, not a hack.  At the moment, the precedence which
favors pointers is a hack, but we cannot remove it due to powerpc being a
important platform.  Also, favoring the pointers may be useful for other
similar to ppc platforms even if we are not aware of them right now.

That said, conditioning the precedence on (!reload_in_progress &&
!reload_completed) fixes the bug so I consider this to be the second best thing
to a hook.

It is not as good as a hook because there may be corner cases during reload
when pointers should be favored on powerpc (multi-dimension array references,
probably).  On the other hand m68k need unrestricted (in the sense that all
RTX_OBJ are the same) commutativeness during reload to avoid ICEs.

So, conditioning pointer precedence on reload_in_progress has possible
performance degradation on powerpc on one hand and ICE on m68k on the other.

> I'm pretty sure that we are talking about two different things. :-)

Err, I don't fully follow you here.  What are the two different things?


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (11 preceding siblings ...)
  2009-06-23 17:22 ` mkuvyrkov at gcc dot gnu dot org
@ 2009-06-24 16:02 ` mkuvyrkov at gcc dot gnu dot org
  2009-08-04 12:42 ` rguenth at gcc dot gnu dot org
                   ` (8 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: mkuvyrkov at gcc dot gnu dot org @ 2009-06-24 16:02 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #13 from mkuvyrkov at gcc dot gnu dot org  2009-06-24 16:02 -------
Created an attachment (id=18061)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18061&action=view)
Proposed patch

Here is a patch moving precedence handling of pointers to powerpc backend.


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (12 preceding siblings ...)
  2009-06-24 16:02 ` mkuvyrkov at gcc dot gnu dot org
@ 2009-08-04 12:42 ` rguenth at gcc dot gnu dot org
  2009-08-04 13:14 ` mkuvyrkov at gcc dot gnu dot org
                   ` (7 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2009-08-04 12:42 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #14 from rguenth at gcc dot gnu dot org  2009-08-04 12:29 -------
GCC 4.3.4 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.4                       |4.3.5


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (13 preceding siblings ...)
  2009-08-04 12:42 ` rguenth at gcc dot gnu dot org
@ 2009-08-04 13:14 ` mkuvyrkov at gcc dot gnu dot org
  2009-08-04 13:36 ` bergner at gcc dot gnu dot org
                   ` (6 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: mkuvyrkov at gcc dot gnu dot org @ 2009-08-04 13:14 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #15 from mkuvyrkov at gcc dot gnu dot org  2009-08-04 13:14 -------
There are several (4, I think) patches posted in gcc-patches@ for this bug.  A
reload/recog maintainer is needed to choose the most appropriate one.


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (14 preceding siblings ...)
  2009-08-04 13:14 ` mkuvyrkov at gcc dot gnu dot org
@ 2009-08-04 13:36 ` bergner at gcc dot gnu dot org
  2009-08-04 13:43 ` mkuvyrkov at gcc dot gnu dot org
                   ` (5 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: bergner at gcc dot gnu dot org @ 2009-08-04 13:36 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #16 from bergner at gcc dot gnu dot org  2009-08-04 13:35 -------
There have been many patches posted, but most have caused serious performance
degradations on power.  However, the two latest patches to reload do not.  They
are:

 1) http://gcc.gnu.org/ml/gcc-patches/2009-07/msg00816.html
 2) http://gcc.gnu.org/ml/gcc-patches/2009-07/msg00823.html

Maxim, have you tried either of these patches and do they work for you?

Uli, can you please have a look at Richard's and Paolo's patches and does one
or the other seem like a "better" fix?


-- 

bergner at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |uweigand at gcc dot gnu dot
                   |                            |org


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (15 preceding siblings ...)
  2009-08-04 13:36 ` bergner at gcc dot gnu dot org
@ 2009-08-04 13:43 ` mkuvyrkov at gcc dot gnu dot org
  2009-08-05 14:59 ` uweigand at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: mkuvyrkov at gcc dot gnu dot org @ 2009-08-04 13:43 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #17 from mkuvyrkov at gcc dot gnu dot org  2009-08-04 13:43 -------
I'll try the above two patches and will report in a couple of days.


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (16 preceding siblings ...)
  2009-08-04 13:43 ` mkuvyrkov at gcc dot gnu dot org
@ 2009-08-05 14:59 ` uweigand at gcc dot gnu dot org
  2009-08-10 15:34 ` uweigand at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: uweigand at gcc dot gnu dot org @ 2009-08-05 14:59 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #18 from uweigand at gcc dot gnu dot org  2009-08-05 14:59 -------
(In reply to comment #16)
> Uli, can you please have a look at Richard's and Paolo's patches and does one
> or the other seem like a "better" fix?

I've yet another suggestion :-)   See my message at:
http://gcc.gnu.org/ml/gcc-patches/2009-08/msg00254.html


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (17 preceding siblings ...)
  2009-08-05 14:59 ` uweigand at gcc dot gnu dot org
@ 2009-08-10 15:34 ` uweigand at gcc dot gnu dot org
  2009-08-23 18:19 ` schwab at linux-m68k dot org
                   ` (2 subsequent siblings)
  21 siblings, 0 replies; 31+ messages in thread
From: uweigand at gcc dot gnu dot org @ 2009-08-10 15:34 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #19 from uweigand at gcc dot gnu dot org  2009-08-10 15:34 -------
Subject: Bug 37053

Author: uweigand
Date: Mon Aug 10 15:34:09 2009
New Revision: 150626

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=150626
Log:
        PR target/37053
        * reload1.c (reload_as_needed): Use cancel_changes to completely
        undo a failed replacement attempt.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/reload1.c


-- 


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


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

* [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (18 preceding siblings ...)
  2009-08-10 15:34 ` uweigand at gcc dot gnu dot org
@ 2009-08-23 18:19 ` schwab at linux-m68k dot org
  2009-12-29 19:30 ` [Bug middle-end/37053] [4.3/4.4 " bonzini at gnu dot org
  2010-05-22 18:25 ` rguenth at gcc dot gnu dot org
  21 siblings, 0 replies; 31+ messages in thread
From: schwab at linux-m68k dot org @ 2009-08-23 18:19 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #20 from schwab at linux-m68k dot org  2009-08-23 18:19 -------
Fixed in 4.5.0.


-- 

schwab at linux-m68k dot org changed:

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


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (19 preceding siblings ...)
  2009-08-23 18:19 ` schwab at linux-m68k dot org
@ 2009-12-29 19:30 ` bonzini at gnu dot org
  2010-05-22 18:25 ` rguenth at gcc dot gnu dot org
  21 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2009-12-29 19:30 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #21 from bonzini at gnu dot org  2009-12-29 19:30 -------
Reopening since it is still broken on the other open branches.


-- 

bonzini at gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
      Known to work|                            |4.5.0
         Resolution|FIXED                       |
            Summary|[4.3/4.4/4.5 regression] ICE|[4.3/4.4 regression] ICE in
                   |in                          |reload_cse_simplify_operands
                   |reload_cse_simplify_operands|, at postreload.c:395
                   |, at postreload.c:395       |


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
  2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
                   ` (20 preceding siblings ...)
  2009-12-29 19:30 ` [Bug middle-end/37053] [4.3/4.4 " bonzini at gnu dot org
@ 2010-05-22 18:25 ` rguenth at gcc dot gnu dot org
  21 siblings, 0 replies; 31+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2010-05-22 18:25 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #22 from rguenth at gcc dot gnu dot org  2010-05-22 18:12 -------
GCC 4.3.5 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.5                       |4.3.6


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


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
                   ` (6 preceding siblings ...)
  2010-11-06 16:25 ` tg at mirbsd dot org
@ 2011-06-27 14:26 ` rguenth at gcc dot gnu.org
  7 siblings, 0 replies; 31+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-06-27 14:26 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.6                       |4.4.7

--- Comment #30 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-06-27 12:13:51 UTC ---
4.3 branch is being closed, moving to 4.4.7 target.


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
                   ` (5 preceding siblings ...)
  2010-11-03 21:42 ` tg at mirbsd dot org
@ 2010-11-06 16:25 ` tg at mirbsd dot org
  2011-06-27 14:26 ` rguenth at gcc dot gnu.org
  7 siblings, 0 replies; 31+ messages in thread
From: tg at mirbsd dot org @ 2010-11-06 16:25 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #29 from Thorsten Glaser <tg at mirbsd dot org> 2010-11-06 16:25:00 UTC ---
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=602099#20
in case someone’s interested…


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2010-11-01 16:58 ` bonzini at gnu dot org
@ 2010-11-03 21:42 ` tg at mirbsd dot org
  2010-11-06 16:25 ` tg at mirbsd dot org
  2011-06-27 14:26 ` rguenth at gcc dot gnu.org
  7 siblings, 0 replies; 31+ messages in thread
From: tg at mirbsd dot org @ 2010-11-03 21:42 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #28 from Thorsten Glaser <tg at mirbsd dot org> 2010-11-03 21:41:38 UTC ---
OK, added.


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2010-11-01 16:54 ` tg at mirbsd dot org
@ 2010-11-01 16:58 ` bonzini at gnu dot org
  2010-11-03 21:42 ` tg at mirbsd dot org
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2010-11-01 16:58 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #27 from Paolo Bonzini <bonzini at gnu dot org> 2010-11-01 16:58:20 UTC ---
Better: not for this testcase.  We found it on CRIS, but the bug could really
happen on any target.


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2010-11-01 16:40 ` bonzini at gnu dot org
@ 2010-11-01 16:54 ` tg at mirbsd dot org
  2010-11-01 16:58 ` bonzini at gnu dot org
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 31+ messages in thread
From: tg at mirbsd dot org @ 2010-11-01 16:54 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #26 from Thorsten Glaser <tg at mirbsd dot org> 2010-11-01 16:53:18 UTC ---
Probably, but not on m68k ;-)


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
  2010-11-01 16:11 ` tg at mirbsd dot org
  2010-11-01 16:39 ` bonzini at gnu dot org
@ 2010-11-01 16:40 ` bonzini at gnu dot org
  2010-11-01 16:54 ` tg at mirbsd dot org
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2010-11-01 16:40 UTC (permalink / raw)
  To: gcc-bugs

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

Paolo Bonzini <bonzini at gnu dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nospamname at web dot de

--- Comment #25 from Paolo Bonzini <bonzini at gnu dot org> 2010-11-01 16:40:07 UTC ---
*** Bug 40414 has been marked as a duplicate of this bug. ***


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
  2010-11-01 16:11 ` tg at mirbsd dot org
@ 2010-11-01 16:39 ` bonzini at gnu dot org
  2010-11-01 16:40 ` bonzini at gnu dot org
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 31+ messages in thread
From: bonzini at gnu dot org @ 2010-11-01 16:39 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #24 from Paolo Bonzini <bonzini at gnu dot org> 2010-11-01 16:38:53 UTC ---
You'd need also the patch for bug 41064.


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

* [Bug middle-end/37053] [4.3/4.4 regression] ICE in reload_cse_simplify_operands, at postreload.c:395
       [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
@ 2010-11-01 16:11 ` tg at mirbsd dot org
  2010-11-01 16:39 ` bonzini at gnu dot org
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 31+ messages in thread
From: tg at mirbsd dot org @ 2010-11-01 16:11 UTC (permalink / raw)
  To: gcc-bugs

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

Thorsten Glaser <tg at mirbsd dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tg at mirbsd dot org

--- Comment #23 from Thorsten Glaser <tg at mirbsd dot org> 2010-11-01 16:10:32 UTC ---
Backported r150626 from Comment 19 and applied to Debian/m68k gcc-4.4
(native). Fixes this ICE when building libjpeg for me. No regression
tests run, though.


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

end of thread, other threads:[~2011-06-27 14:26 UTC | newest]

Thread overview: 31+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-08-08  1:02 [Bug c/37053] New: ICE in reload_cse_simplify_operands, at postreload.c:395 stephen at marenka dot net
2008-08-08  1:03 ` [Bug c/37053] " stephen at marenka dot net
2008-08-08  1:04 ` stephen at marenka dot net
2008-09-11 15:21 ` [Bug middle-end/37053] [4.3/4.4 regression] " schwab at suse dot de
2008-09-11 15:30 ` bonzini at gnu dot org
2008-09-16 21:01 ` schwab at suse dot de
2008-09-16 21:03 ` bonzini at gnu dot org
2008-09-20 14:30 ` rguenth at gcc dot gnu dot org
2009-06-23 11:40 ` [Bug middle-end/37053] [4.3/4.4/4.5 " mkuvyrkov at gcc dot gnu dot org
2009-06-23 12:05 ` bonzini at gnu dot org
2009-06-23 12:26 ` mkuvyrkov at gcc dot gnu dot org
2009-06-23 13:09 ` bonzini at gnu dot org
2009-06-23 17:22 ` mkuvyrkov at gcc dot gnu dot org
2009-06-24 16:02 ` mkuvyrkov at gcc dot gnu dot org
2009-08-04 12:42 ` rguenth at gcc dot gnu dot org
2009-08-04 13:14 ` mkuvyrkov at gcc dot gnu dot org
2009-08-04 13:36 ` bergner at gcc dot gnu dot org
2009-08-04 13:43 ` mkuvyrkov at gcc dot gnu dot org
2009-08-05 14:59 ` uweigand at gcc dot gnu dot org
2009-08-10 15:34 ` uweigand at gcc dot gnu dot org
2009-08-23 18:19 ` schwab at linux-m68k dot org
2009-12-29 19:30 ` [Bug middle-end/37053] [4.3/4.4 " bonzini at gnu dot org
2010-05-22 18:25 ` rguenth at gcc dot gnu dot org
     [not found] <bug-37053-4@http.gcc.gnu.org/bugzilla/>
2010-11-01 16:11 ` tg at mirbsd dot org
2010-11-01 16:39 ` bonzini at gnu dot org
2010-11-01 16:40 ` bonzini at gnu dot org
2010-11-01 16:54 ` tg at mirbsd dot org
2010-11-01 16:58 ` bonzini at gnu dot org
2010-11-03 21:42 ` tg at mirbsd dot org
2010-11-06 16:25 ` tg at mirbsd dot org
2011-06-27 14:26 ` 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).