public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/25335]  New: reload leaves insns from earlier passes around: fatal for postinc
@ 2005-12-10  3:57 hp at gcc dot gnu dot org
  2005-12-10  4:00 ` [Bug middle-end/25335] [4.1/4.2 Regression] " pinskia at gcc dot gnu dot org
                   ` (22 more replies)
  0 siblings, 23 replies; 24+ messages in thread
From: hp at gcc dot gnu dot org @ 2005-12-10  3:57 UTC (permalink / raw)
  To: gcc-bugs

See assembly output for gcc.target/cris/rld-legit2.c (here for cris-axis-elf):
_g:
        subq 4,$sp
        move $srp,[$sp]
        subq 40,$sp
        movem $r8,[$sp]
        move $r10,$srp
        move.d $r11,[$sp+36]
        move $srp,$r9
        addq 2,$r9
        move $r9,$srp
        subq 2,$r9
        move.w [$r9+],$r11
        move.w [$r9+],$r12
        movs.w $r12,$r12
        move.d [$sp+36],$r13
        clear.b [$r12+$r13.b]
        move $srp,$r10
        movem [$sp+],$r8
        addq 4,$sp
        jump [$sp+]

Note the two post-increments; the second one is used and gets the wrong value.
(A failing executable test-case can trivially be constructed and will be.)
Reload seems to need two rounds, but the emitted reload insns for each pass
is left around. This is exposed but not actually caused by the fix for
PR middle-end/24912.


-- 
           Summary: reload leaves insns from earlier passes around: fatal
                    for postinc
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Keywords: wrong-code
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hp at gcc dot gnu dot org
GCC target triplet: cris-*-*


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
@ 2005-12-10  4:00 ` pinskia at gcc dot gnu dot org
  2005-12-10  4:53 ` hp at gcc dot gnu dot org
                   ` (21 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-12-10  4:00 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from pinskia at gcc dot gnu dot org  2005-12-10 04:00 -------
So this is a latent bug exposed by that fix which means this should be marked
as a regression.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|reload leaves insns from    |[4.1/4.2 Regression] reload
                   |earlier passes around: fatal|leaves insns from earlier
                   |for postinc                 |passes around: fatal for
                   |                            |postinc
   Target Milestone|---                         |4.1.0


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
  2005-12-10  4:00 ` [Bug middle-end/25335] [4.1/4.2 Regression] " pinskia at gcc dot gnu dot org
@ 2005-12-10  4:53 ` hp at gcc dot gnu dot org
  2005-12-19 19:26 ` mmitchel at gcc dot gnu dot org
                   ` (20 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: hp at gcc dot gnu dot org @ 2005-12-10  4:53 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from hp at gcc dot gnu dot org  2005-12-10 04:53 -------
I mean for *this* test-case it's exposed by that fix.  The leaving-behind
of reload-insns from earlier rounds for each new reload round is general.


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
  2005-12-10  4:00 ` [Bug middle-end/25335] [4.1/4.2 Regression] " pinskia at gcc dot gnu dot org
  2005-12-10  4:53 ` hp at gcc dot gnu dot org
@ 2005-12-19 19:26 ` mmitchel at gcc dot gnu dot org
  2005-12-19 20:10 ` hp at gcc dot gnu dot org
                   ` (19 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2005-12-19 19:26 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from mmitchel at gcc dot gnu dot org  2005-12-19 19:26 -------
Do we have a testcase for this bug on a primary/secondary platform, or is this
something CRIS-specific?


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (2 preceding siblings ...)
  2005-12-19 19:26 ` mmitchel at gcc dot gnu dot org
@ 2005-12-19 20:10 ` hp at gcc dot gnu dot org
  2006-01-15 22:11 ` mmitchel at gcc dot gnu dot org
                   ` (18 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: hp at gcc dot gnu dot org @ 2005-12-19 20:10 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #4 from hp at gcc dot gnu dot org  2005-12-19 20:10 -------
Incomplete analysis, can't reply to the second part of comment #3.
"No" to the first part.


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (3 preceding siblings ...)
  2005-12-19 20:10 ` hp at gcc dot gnu dot org
@ 2006-01-15 22:11 ` mmitchel at gcc dot gnu dot org
  2006-01-30 20:24 ` amylaar at gcc dot gnu dot org
                   ` (17 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2006-01-15 22:11 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #5 from mmitchel at gcc dot gnu dot org  2006-01-15 22:11 -------
I'm leaving this at P3 until we have more information.  Clearly, if this is a
general problem that affets primary/secondary targets, we should increase the
priority.


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (4 preceding siblings ...)
  2006-01-15 22:11 ` mmitchel at gcc dot gnu dot org
@ 2006-01-30 20:24 ` amylaar at gcc dot gnu dot org
  2006-01-30 21:53 ` hp at gcc dot gnu dot org
                   ` (16 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at gcc dot gnu dot org @ 2006-01-30 20:24 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #6 from amylaar at gcc dot gnu dot org  2006-01-30 20:24 -------
> Reload seems to need two rounds, but the emitted reload insns for each pass
> is left around. This is exposed but not actually caused by the fix for
> PR middle-end/24912.

Reload should be only called once per function compilation.  It iterates till
it
has a stable set of spills and reloads, and only then does it emit reload
insns.
So what do you mean with reload needing two rounds?


-- 

amylaar at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |joern dot rennecke at st dot
                   |                            |com


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (5 preceding siblings ...)
  2006-01-30 20:24 ` amylaar at gcc dot gnu dot org
@ 2006-01-30 21:53 ` hp at gcc dot gnu dot org
  2006-01-31 16:32 ` amylaar at gcc dot gnu dot org
                   ` (15 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: hp at gcc dot gnu dot org @ 2006-01-30 21:53 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #7 from hp at gcc dot gnu dot org  2006-01-30 21:53 -------
Re: "needing two rounds".  Looks like you're hung up on my choice of words.
I suggest ignore that and instead just run the test-case.


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (6 preceding siblings ...)
  2006-01-30 21:53 ` hp at gcc dot gnu dot org
@ 2006-01-31 16:32 ` amylaar at gcc dot gnu dot org
  2006-01-31 20:19 ` amylaar at gcc dot gnu dot org
                   ` (14 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at gcc dot gnu dot org @ 2006-01-31 16:32 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #8 from amylaar at gcc dot gnu dot org  2006-01-31 16:32 -------
(In reply to comment #7)
> Re: "needing two rounds".  Looks like you're hung up on my choice of words.
> I suggest ignore that and instead just run the test-case.
> 

The patch for PR middle-end/24912 actually caused this regression.  I have
added a reply there explaining the problem.


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (7 preceding siblings ...)
  2006-01-31 16:32 ` amylaar at gcc dot gnu dot org
@ 2006-01-31 20:19 ` amylaar at gcc dot gnu dot org
  2006-01-31 22:10 ` hp at gcc dot gnu dot org
                   ` (13 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at gcc dot gnu dot org @ 2006-01-31 20:19 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #9 from amylaar at gcc dot gnu dot org  2006-01-31 20:19 -------
(In reply to comment #0)
Visual inspection indicates that this patch fixes the testcase:
http://gcc.gnu.org/ml/gcc-patches/2006-01/msg02229.html
Can you confirm?


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (8 preceding siblings ...)
  2006-01-31 20:19 ` amylaar at gcc dot gnu dot org
@ 2006-01-31 22:10 ` hp at gcc dot gnu dot org
  2006-01-31 22:32 ` amylaar at gcc dot gnu dot org
                   ` (12 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: hp at gcc dot gnu dot org @ 2006-01-31 22:10 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #10 from hp at gcc dot gnu dot org  2006-01-31 22:10 -------
The bug is (unfortunately?) not exposed with trunk revision 110246.

Confirmed that the patch referred to in comment #9 fixes the bug
when applied to trunk revision 108225 (from around Dec  8).

Confirmed also that the patch fixes the bug when applied to gcc-4_1-branch
revision 110348.
Nice, thanks!


-- 

hp at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |patch
   Last reconfirmed|0000-00-00 00:00:00         |2006-01-31 22:10:45
               date|                            |


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (9 preceding siblings ...)
  2006-01-31 22:10 ` hp at gcc dot gnu dot org
@ 2006-01-31 22:32 ` amylaar at gcc dot gnu dot org
  2006-02-10 17:42 ` amylaar at gcc dot gnu dot org
                   ` (11 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at gcc dot gnu dot org @ 2006-01-31 22:32 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #11 from amylaar at gcc dot gnu dot org  2006-01-31 22:32 -------
(In reply to comment #9)

P.S.: For the 4.1 branch and older release branches, it is sufficient to swap
the order of the calls to emit_reload_insns and subst_reloads.



-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (10 preceding siblings ...)
  2006-01-31 22:32 ` amylaar at gcc dot gnu dot org
@ 2006-02-10 17:42 ` amylaar at gcc dot gnu dot org
  2006-02-10 22:11 ` hp at gcc dot gnu dot org
                   ` (10 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at gcc dot gnu dot org @ 2006-02-10 17:42 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #12 from amylaar at gcc dot gnu dot org  2006-02-10 17:42 -------
I get the following code for rld-legit2.c with the patch I am currently
testing:
_g:
        subq 4,$sp
        move $srp,[$sp]
        subq 40,$sp
        movem $r8,[$sp]
        move $r10,$srp
        move.d $r11,[$sp+36]
        move $srp,$r13
        addq 2,$r13
        move $r13,$srp
        subq 2,$r13
        movs.w [$r13+],$r9
        move.d [$sp+36],$r13
        clear.b [$r9+$r13.b]
        move $srp,$r10
        movem [$sp+],$r8
        addq 4,$sp
        jump [$sp+]
        .size   _g, .-_g

I suppose this is right.


-- 

amylaar at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |amylaar at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-01-31 22:10:45         |2006-02-10 17:42:38
               date|                            |


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (11 preceding siblings ...)
  2006-02-10 17:42 ` amylaar at gcc dot gnu dot org
@ 2006-02-10 22:11 ` hp at gcc dot gnu dot org
  2006-02-13 15:33 ` amylaar at gcc dot gnu dot org
                   ` (9 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: hp at gcc dot gnu dot org @ 2006-02-10 22:11 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #13 from hp at gcc dot gnu dot org  2006-02-10 22:11 -------
(In reply to comment #12)
This code is exactly the same as the correct code I saw
for cris-axis-linux-gnu with the patch in comment #9, so I can confirm that
the emitted code is correct by visual inspection.  On the other hand,
the bug is not exposed on trunk version 110246 or 110787 without the patch.

(For the record, the code is clearly sub-optimal,
but that's not the issue here.)

Again, thanks for fixing this (apparently) long-standing wart.


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (12 preceding siblings ...)
  2006-02-10 22:11 ` hp at gcc dot gnu dot org
@ 2006-02-13 15:33 ` amylaar at gcc dot gnu dot org
  2006-02-14  9:17 ` mmitchel at gcc dot gnu dot org
                   ` (8 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at gcc dot gnu dot org @ 2006-02-13 15:33 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #14 from amylaar at gcc dot gnu dot org  2006-02-13 15:33 -------
Subject: Bug 25335

Author: amylaar
Date: Mon Feb 13 15:32:57 2006
New Revision: 110916

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110916
Log:
Add PR marker for:
        PR middle-end/25335
        * reload1.c (gen_reload): Also use find_replacement in UNARY_P case.

Modified:
    trunk/gcc/ChangeLog


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (13 preceding siblings ...)
  2006-02-13 15:33 ` amylaar at gcc dot gnu dot org
@ 2006-02-14  9:17 ` mmitchel at gcc dot gnu dot org
  2006-02-14 20:03 ` joern dot rennecke at st dot com
                   ` (7 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2006-02-14  9:17 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #15 from mmitchel at gcc dot gnu dot org  2006-02-14 09:17 -------
OK for 4.1, should you wish to apply it there.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P5


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (14 preceding siblings ...)
  2006-02-14  9:17 ` mmitchel at gcc dot gnu dot org
@ 2006-02-14 20:03 ` joern dot rennecke at st dot com
  2006-02-15  1:56 ` mark at codesourcery dot com
                   ` (6 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: joern dot rennecke at st dot com @ 2006-02-14 20:03 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #16 from joern dot rennecke at st dot com  2006-02-14 20:03 -------
Subject: Re:  [4.1/4.2 Regression] reload leaves insns from earlier passes
around: fatal for postinc

mmitchel at gcc dot gnu dot org wrote:

>------- Comment #15 from mmitchel at gcc dot gnu dot org  2006-02-14 09:17 -------
>OK for 4.1, should you wish to apply it there.
>
>
>  
>
I have started regression tests for i686-pc-linux-gnu native, X arm-elf, 
X cris-elf, X sh-elf and
X sh64-elf, but so far only the arm-elf and cris-elf tests are done.  Is 
it still OK to check the patch in
tomorrow after the freeze?


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (15 preceding siblings ...)
  2006-02-14 20:03 ` joern dot rennecke at st dot com
@ 2006-02-15  1:56 ` mark at codesourcery dot com
  2006-02-15 13:28 ` amylaar at gcc dot gnu dot org
                   ` (5 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: mark at codesourcery dot com @ 2006-02-15  1:56 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #17 from mark at codesourcery dot com  2006-02-15 01:56 -------
Subject: Re:  [4.1/4.2 Regression] reload leaves insns
 from earlier passes around: fatal for postinc

Joern RENNECKE wrote:

> I have started regression tests for i686-pc-linux-gnu native, X arm-elf,
> X cris-elf, X sh-elf and
> X sh64-elf, but so far only the arm-elf and cris-elf tests are done.  Is
> it still OK to check the patch in
> tomorrow after the freeze?

Yes.  (I will give a heads-up before I actually start to build a release
candidate.)


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (16 preceding siblings ...)
  2006-02-15  1:56 ` mark at codesourcery dot com
@ 2006-02-15 13:28 ` amylaar at gcc dot gnu dot org
  2006-02-28 20:42 ` mmitchel at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at gcc dot gnu dot org @ 2006-02-15 13:28 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #18 from amylaar at gcc dot gnu dot org  2006-02-15 13:28 -------
Subject: Bug 25335

Author: amylaar
Date: Wed Feb 15 13:28:05 2006
New Revision: 111111

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=111111
Log:
        PR middle-end/25335
        * reload1.c (gen_reload): Also use find_replacement in UNARY_P case.

Modified:
    branches/gcc-4_1-branch/gcc/ChangeLog
    branches/gcc-4_1-branch/gcc/reload1.c


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (17 preceding siblings ...)
  2006-02-15 13:28 ` amylaar at gcc dot gnu dot org
@ 2006-02-28 20:42 ` mmitchel at gcc dot gnu dot org
  2006-05-25  2:46 ` mmitchel at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2006-02-28 20:42 UTC (permalink / raw)
  To: gcc-bugs



-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.0                       |4.1.1


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (18 preceding siblings ...)
  2006-02-28 20:42 ` mmitchel at gcc dot gnu dot org
@ 2006-05-25  2:46 ` mmitchel at gcc dot gnu dot org
  2006-07-20 11:13 ` pinskia at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  22 siblings, 0 replies; 24+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2006-05-25  2:46 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #19 from mmitchel at gcc dot gnu dot org  2006-05-25 02:36 -------
Will not be fixed in 4.1.1; adjust target milestone to 4.1.2.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.1                       |4.1.2


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (19 preceding siblings ...)
  2006-05-25  2:46 ` mmitchel at gcc dot gnu dot org
@ 2006-07-20 11:13 ` pinskia at gcc dot gnu dot org
  2006-07-20 17:09 ` amylaar at spamcop dot net
  2006-07-20 22:14 ` pinskia at gcc dot gnu dot org
  22 siblings, 0 replies; 24+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2006-07-20 11:13 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #20 from pinskia at gcc dot gnu dot org  2006-07-20 11:13 -------
This was fixed, correct?


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (20 preceding siblings ...)
  2006-07-20 11:13 ` pinskia at gcc dot gnu dot org
@ 2006-07-20 17:09 ` amylaar at spamcop dot net
  2006-07-20 22:14 ` pinskia at gcc dot gnu dot org
  22 siblings, 0 replies; 24+ messages in thread
From: amylaar at spamcop dot net @ 2006-07-20 17:09 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #21 from amylaar at spamcop dot net  2006-07-20 17:08 -------
Subject: Re:  [4.1/4.2 Regression] reload leaves
        insns from earlier passes around: fatal for postinc

Quoting pinskia at gcc dot gnu dot org <gcc-bugzilla@gcc.gnu.org>:

>
>
> ------- Comment #20 from pinskia at gcc dot gnu dot org  2006-07-20 11:13
> -------
> This was fixed, correct?

Yes, on mainline and 4.1.


-- 


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
* [Bug middle-end/25335] [4.1/4.2 Regression] reload leaves insns from earlier passes around: fatal for postinc
  2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
                   ` (21 preceding siblings ...)
  2006-07-20 17:09 ` amylaar at spamcop dot net
@ 2006-07-20 22:14 ` pinskia at gcc dot gnu dot org
  22 siblings, 0 replies; 24+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2006-07-20 22:14 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #22 from pinskia at gcc dot gnu dot org  2006-07-20 22:14 -------
Fixed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|4.1.2                       |4.1.1


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


^ permalink raw reply	[flat|nested] 24+ messages in thread
end of thread, other threads:[~2006-07-20 22:14 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-12-10  3:57 [Bug middle-end/25335] New: reload leaves insns from earlier passes around: fatal for postinc hp at gcc dot gnu dot org
2005-12-10  4:00 ` [Bug middle-end/25335] [4.1/4.2 Regression] " pinskia at gcc dot gnu dot org
2005-12-10  4:53 ` hp at gcc dot gnu dot org
2005-12-19 19:26 ` mmitchel at gcc dot gnu dot org
2005-12-19 20:10 ` hp at gcc dot gnu dot org
2006-01-15 22:11 ` mmitchel at gcc dot gnu dot org
2006-01-30 20:24 ` amylaar at gcc dot gnu dot org
2006-01-30 21:53 ` hp at gcc dot gnu dot org
2006-01-31 16:32 ` amylaar at gcc dot gnu dot org
2006-01-31 20:19 ` amylaar at gcc dot gnu dot org
2006-01-31 22:10 ` hp at gcc dot gnu dot org
2006-01-31 22:32 ` amylaar at gcc dot gnu dot org
2006-02-10 17:42 ` amylaar at gcc dot gnu dot org
2006-02-10 22:11 ` hp at gcc dot gnu dot org
2006-02-13 15:33 ` amylaar at gcc dot gnu dot org
2006-02-14  9:17 ` mmitchel at gcc dot gnu dot org
2006-02-14 20:03 ` joern dot rennecke at st dot com
2006-02-15  1:56 ` mark at codesourcery dot com
2006-02-15 13:28 ` amylaar at gcc dot gnu dot org
2006-02-28 20:42 ` mmitchel at gcc dot gnu dot org
2006-05-25  2:46 ` mmitchel at gcc dot gnu dot org
2006-07-20 11:13 ` pinskia at gcc dot gnu dot org
2006-07-20 17:09 ` amylaar at spamcop dot net
2006-07-20 22:14 ` pinskia at gcc dot gnu dot 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).