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

[PRISM] Don't allocate labels when not needed

Browse Source 
The labels lstart, lend, lcont are only needed when there is a rescue
clause. They are not needed when there is only an ensure clause or
neither.
This commit is contained in:
Peter Zhu 2024-01-16 11:09:00 -05:00 committed by Kevin Newton
parent 1caa881a56
commit 70a8ed0775
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
prism_compile.c
View File

@ -3201,12 +3201,11 @@ pm_compile_node(rb_iseq_t *iseq, const pm_node_t *node, LINK_ANCHOR *const ret,
case PM_BEGIN_NODE: { case PM_BEGIN_NODE: {
pm_begin_node_t *begin_node = (pm_begin_node_t *) node; pm_begin_node_t *begin_node = (pm_begin_node_t *) node;
rb_iseq_t *child_iseq; rb_iseq_t *child_iseq;
LABEL *lstart = NEW_LABEL(lineno);
LABEL *lend = NEW_LABEL(lineno);
LABEL *lcont = NEW_LABEL(lineno);
if (begin_node->rescue_clause) { if (begin_node->rescue_clause) {
LABEL *lstart = NEW_LABEL(lineno);
LABEL *lend = NEW_LABEL(lineno);
LABEL *lcont = NEW_LABEL(lineno);
pm_scope_node_t rescue_scope_node; pm_scope_node_t rescue_scope_node;
pm_scope_node_init((pm_node_t *)begin_node->rescue_clause, &rescue_scope_node, scope_node, parser); pm_scope_node_init((pm_node_t *)begin_node->rescue_clause, &rescue_scope_node, scope_node, parser);
@ -3292,14 +3291,12 @@ pm_compile_node(rb_iseq_t *iseq, const pm_node_t *node, LINK_ANCHOR *const ret,
} }
if (!begin_node->rescue_clause && !begin_node->ensure_clause) { if (!begin_node->rescue_clause && !begin_node->ensure_clause) {
ADD_LABEL(ret, lstart);
if (begin_node->statements) { if (begin_node->statements) {
PM_COMPILE((pm_node_t *)begin_node->statements); PM_COMPILE((pm_node_t *)begin_node->statements);
} }
else { else {
PM_PUTNIL_UNLESS_POPPED; PM_PUTNIL_UNLESS_POPPED;
} }
ADD_LABEL(ret, lend);
} }
return; return;
} }