--- Comment #25 from pinskia at gcc dot gnu dot org 2007-07-27 09:42
---
Fixed.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #24 from spark at gcc dot gnu dot org 2007-07-05 23:44 ---
Subject: Bug 32475
Author: spark
Date: Thu Jul 5 23:44:44 2007
New Revision: 126391
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126391
Log:
2007-07-05 Seongbae Park [EMAIL PROTECTED]
PR
--- Comment #23 from spark at gcc dot gnu dot org 2007-07-03 19:41 ---
This patch:
diff -r 29a7055e69fc gcc/df-scan.c
--- a/gcc/df-scan.c Sun Jun 24 00:16:55 2007 +
+++ b/gcc/df-scan.c Tue Jul 03 12:38:41 2007 -0700
@@ -2763,6 +2763,12 @@ df_def_record_1 (struct
--- Comment #20 from spark at gcc dot gnu dot org 2007-07-02 06:56 ---
We already put stack pointer in the exit block use set always.
However, after epilogue is generated, we see the sp restoration code:
1 NOTE_INSN_DELETED
4 NOTE_INSN_BASIC_BLOCK
22 [--sp:SI]=bp:SI
23
--- Comment #21 from zadeck at naturalbridge dot com 2007-07-02 14:12
---
Subject: Re: [4.3 Regression] function with asm()
does not setup stack frame
ian at airs dot com wrote:
--- Comment #13 from ian at airs dot com 2007-06-30 18:08 ---
The problem here is that
--- Comment #22 from spark at gcc dot gnu dot org 2007-07-02 17:33 ---
(In reply to comment #21)
I don't think this patch is quite enough.
IIUC, it is still possible (although very unlikely) for stack space to be
required without any MEM_LOAD or MEM_STORE present, and without any
--- Comment #16 from zadeck at naturalbridge dot com 2007-07-02 00:19
---
Subject: Re: [4.3 Regression] function with asm()
does not setup stack frame
Ian Lance Taylor wrote:
Adding the stack pointer for asms is certainly the easiest thing to do.
I don't know if that is
--- Comment #17 from ian at airs dot com 2007-07-02 01:45 ---
Before I tackle the specific questions, let me try to explain the issue that I
see. The issue is that during a function the value of the stack pointer
register must always be at or below the local variables. An interrupt
--- Comment #18 from zadeck at naturalbridge dot com 2007-07-02 02:45
---
Subject: Re: [4.3 Regression] function with asm()
does not setup stack frame
ian at airs dot com wrote:
--- Comment #17 from ian at airs dot com 2007-07-02 01:45 ---
Before I tackle the specific
--- Comment #19 from ian at airs dot com 2007-07-02 02:50 ---
I don't see an inconsistency in my answers. If you mark local variables as
having a reference to the stack pointer before combine, then you should handle
log_links as appropriate.
I am explicitly not telling you what to do.
--- Comment #9 from marcus at jet dot franken dot de 2007-06-30 08:05
---
This bug has resurfaced in the last two days.
r126139 is affected.
--
marcus at jet dot franken dot de changed:
What|Removed |Added
--- Comment #10 from zadeck at naturalbridge dot com 2007-06-30 11:47
---
Richard,
Could you check to see if this bug is collateral damage from your latest fix to
deletable_insn_p. It's appearance has been tied to that function in the past.
kenny
--
zadeck at naturalbridge
--- Comment #11 from richard at codesourcery dot com 2007-06-30 12:19
---
Subject: Re: [4.3 Regression] function with asm() does not setup stack frame
zadeck at naturalbridge dot com [EMAIL PROTECTED] writes:
Could you check to see if this bug is collateral damage from your
latest
--- Comment #12 from richard at codesourcery dot com 2007-06-30 12:58
---
Subject: Re: [4.3 Regression] function with asm() does not setup stack frame
richard at codesourcery dot com [EMAIL PROTECTED] writes:
zadeck at naturalbridge dot com [EMAIL PROTECTED] writes:
Could you check
--- Comment #13 from ian at airs dot com 2007-06-30 18:08 ---
The problem here is that although the stack pointer is not used in the
function, adjusting it does reserve space on the stack for the local variables
which are used. The local variables are accessed via the frame pointer
--- Comment #14 from zadeck at naturalbridge dot com 2007-06-30 20:11
---
Subject: Re: [4.3 Regression] function with asm()
does not setup stack frame
ian at airs dot com wrote:
--- Comment #13 from ian at airs dot com 2007-06-30 18:08 ---
The problem here is that
--- Comment #15 from iant at google dot com 2007-07-01 01:58 ---
Subject: Re: [4.3 Regression] function with asm() does not setup stack frame
Adding the stack pointer for asms is certainly the easiest thing to do.
I don't know if that is enough. Maybe it is, maybe it isn't. You
--- Comment #8 from marcus at jet dot franken dot de 2007-06-24 10:41
---
is fixed in current SVN.
--
marcus at jet dot franken dot de changed:
What|Removed |Added
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
CC||rguenth at gcc dot gnu dot
|
--- Comment #2 from pinskia at gcc dot gnu dot org 2007-06-23 22:31 ---
The target might have forgot a barrior. the RTL is correct after pro_epilogue.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32475
--- Comment #3 from pinskia at gcc dot gnu dot org 2007-06-23 22:32 ---
dse2 removes the decrement:
(insn/f 24 23 25 2 t2.c:2 (parallel [
(set (reg/f:SI 7 sp)
(plus:SI (reg/f:SI 7 sp)
(const_int -16 [0xfff0])))
--- Comment #4 from spark at gcc dot gnu dot org 2007-06-24 01:05 ---
I think Kenny's last patch for PR32437 fixes this as well.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32475
--- Comment #5 from zadeck at naturalbridge dot com 2007-06-24 01:10
---
Subject: Re: [4.3 Regression] function with asm()
does not setup stack frame
spark at gcc dot gnu dot org wrote:
--- Comment #4 from spark at gcc dot gnu dot org 2007-06-24 01:05 ---
I think Kenny's
--- Comment #6 from spark at gcc dot gnu dot org 2007-06-24 02:01 ---
(In reply to comment #5)
Subject: Re: [4.3 Regression] function with asm()
does not setup stack frame
spark at gcc dot gnu dot org wrote:
--- Comment #4 from spark at gcc dot gnu dot org 2007-06-24 01:05
--- Comment #7 from zadeck at naturalbridge dot com 2007-06-24 02:48
---
Subject: Re: [4.3 Regression] function with asm()
does not setup stack frame
spark at gcc dot gnu dot org wrote:
--- Comment #6 from spark at gcc dot gnu dot org 2007-06-24 02:01 ---
(In reply to
25 matches
Mail list logo
