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

should not restore builtin_inline_index

Browse Source 
`builtin_inline_index` is restored because THEN clause on
`Primitive.mandatory_only?` was compiled twice.

However, f29c9d6d36 skips to compile THEN clause so we don't
need to restore `builtin_inline_index`.
This commit is contained in:
Koichi Sasada 2023-03-17 16:00:34 +09:00
parent fee9044b25
commit e5a5e43c36
Notes: git 2023-03-23 05:03:45 +00:00 
Merged: https://github.com/ruby/ruby/pull/7486
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
compile.c
View File

@ -8348,15 +8348,12 @@ compile_builtin_mandatory_only_method(rb_iseq_t *iseq, const NODE *node, const N
.script_lines = ISEQ_BODY(iseq)->variable.script_lines, .script_lines = ISEQ_BODY(iseq)->variable.script_lines,
}; };
int prev_inline_index = GET_VM()->builtin_inline_index;
ISEQ_BODY(iseq)->mandatory_only_iseq = ISEQ_BODY(iseq)->mandatory_only_iseq =
rb_iseq_new_with_opt(&ast, rb_iseq_base_label(iseq), rb_iseq_new_with_opt(&ast, rb_iseq_base_label(iseq),
rb_iseq_path(iseq), rb_iseq_realpath(iseq), rb_iseq_path(iseq), rb_iseq_realpath(iseq),
nd_line(line_node), NULL, 0, nd_line(line_node), NULL, 0,
ISEQ_TYPE_METHOD, ISEQ_COMPILE_DATA(iseq)->option); ISEQ_TYPE_METHOD, ISEQ_COMPILE_DATA(iseq)->option);
GET_VM()->builtin_inline_index = prev_inline_index;
ALLOCV_END(idtmp); ALLOCV_END(idtmp);
return COMPILE_OK; return COMPILE_OK;
} }