1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

nobody uses the return value of vm_push_frame

Browse Source 
Surprised to see such a waste of time in this super duper hot path.
This commit is contained in:
卜部昌平 2020-07-06 11:51:53 +09:00
parent ba81bc24e6
commit db7f3496dd
Notes: git 2020-07-10 12:24:07 +09:00 
Merged: https://github.com/ruby/ruby/pull/3296
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
vm_insnhelper.c
View File

@ -287,7 +287,7 @@ vm_check_canary(const rb_execution_context_t *ec, VALUE *sp)
#define vm_check_frame(a, b, c, d) #define vm_check_frame(a, b, c, d)
#endif /* VM_CHECK_MODE > 0 */ #endif /* VM_CHECK_MODE > 0 */
static inline rb_control_frame_t * static void
vm_push_frame(rb_execution_context_t *ec, vm_push_frame(rb_execution_context_t *ec,
const rb_iseq_t *iseq, const rb_iseq_t *iseq,
VALUE type, VALUE type,
@ -370,8 +370,6 @@ vm_push_frame(rb_execution_context_t *ec,
} }
} }
#endif #endif
return cfp;
} }
/* return TRUE if the frame is finished */ /* return TRUE if the frame is finished */