This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch committed] Fix target/40313
- From: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
- To: gcc-patches at gcc dot gnu dot org
- Cc: richard dot guenther at gmail dot com
- Date: Wed, 03 Jun 2009 06:19:29 +0900 (JST)
- Subject: Re: [patch committed] Fix target/40313
- References: <20090601.065102.133045611.kkojima@rr.iij4u.or.jp> <84fc9c000905311459y197aaa94j282d444b50fcb27a@mail.gmail.com> <20090601.084826.137524736.kkojima@rr.iij4u.or.jp>
> I'll remove that if line at all.
For the record, I've applied the patch below on trunk after
the tests on sh4-unknown-linux-gnu.
Regards,
kaz
--
2009-06-01 Kaz Kojima <kkojima@gcc.gnu.org>
* config/sh/sh.c: Revert last change.
(sh_expand_epilogue): Emit a blockage insn before the frame
pointer adjustment unconditionally.
--- ORIG/trunk/gcc/config/sh/sh.c 2009-05-31 08:46:05.000000000 +0900
+++ trunk/gcc/config/sh/sh.c 2009-06-01 08:51:21.000000000 +0900
@@ -43,7 +43,6 @@ along with GCC; see the file COPYING3.
#include "target.h"
#include "target-def.h"
#include "real.h"
-#include "debug.h"
#include "langhooks.h"
#include "basic-block.h"
#include "df.h"
@@ -6747,10 +6746,8 @@ sh_expand_epilogue (bool sibcall_p)
if (frame_pointer_needed)
{
- /* We must avoid scheduling the epilogue with previous basic blocks
- when exception handling or frame information is needed.
+ /* We must avoid scheduling the epilogue with previous basic blocks.
See PR/18032 and PR/40313. */
- if (flag_exceptions || dwarf2out_do_frame ())
- emit_insn (gen_blockage ());
+ emit_insn (gen_blockage ());
output_stack_adjust (frame_size, hard_frame_pointer_rtx, e,
&live_regs_mask);