naruse	2013-03-18 11:06:44 +0900 (Mon, 18 Mar 2013)

  New Revision: 39807

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=39807

  Log:
    * vm_exec.h (END_INSN): revert r39517 because the segv seems fixed by
      r39806.

  Modified files:
    trunk/ChangeLog
    trunk/vm_exec.h

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 39806)
+++ ChangeLog	(revision 39807)
@@ -1,3 +1,8 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1
+Mon Mar 18 11:03:23 2013  NARUSE, Yui  <naruse@r...>
+
+	* vm_exec.h (END_INSN): revert r39517 because the segv seems fixed by
+	  r39806.
+
 Mon Mar 18 10:41:06 2013  NARUSE, Yui  <naruse@r...>
 
 	* vm_exec.c: Correct predefined macro name. This typo is introduced by
Index: vm_exec.h
===================================================================
--- vm_exec.h	(revision 39806)
+++ vm_exec.h	(revision 39807)
@@ -116,21 +116,9 @@ error ! https://github.com/ruby/ruby/blob/trunk/vm_exec.h#L116
 
 #endif /* DISPATCH_DIRECT_THREADED_CODE */
 
-#if defined(__llvm__) && !defined(__clang__)
-  /* llvm-gcc may optimize out reg_cfp and cause Stack/cfp consistency error
-   * when the instruction doesn't use reg_cfp.
-   * Usually instructions use PUSH() but for example trace doesn't.
-   * This hack cause speed down but you shouldn't use llvm-gcc, use clang.
-   */
 #define END_INSN(insn)      \
-  { rb_control_frame_t *volatile RB_UNUSED_VAR(tmpcfp) = reg_cfp; } \
   DEBUG_END_INSN();         \
   TC_DISPATCH(insn);
-#else
-#define END_INSN(insn)      \
-  DEBUG_END_INSN();         \
-  TC_DISPATCH(insn);
-#endif
 
 #define INSN_DISPATCH()     \
   TC_DISPATCH(__START__)    \

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/