This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch committed] Fix target/40313


> 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);


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]