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

opt_new needs to happen after safe navigation

Browse Source 
If safe navigation instructions happen first, we get a stack
inconsistency error.
This commit is contained in:
Aaron Patterson 2025-04-29 10:02:00 -07:00 committed by Aaron Patterson
parent e6974be545
commit 203614080f
Notes: git 2025-04-29 20:33:42 +00:00 
Merged: https://github.com/ruby/ruby/pull/13205
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
prism_compile.c
View File

@ -3621,8 +3621,6 @@ pm_compile_call(rb_iseq_t *iseq, const pm_call_node_t *call_node, LINK_ANCHOR *c
const pm_node_location_t location = PM_LOCATION_START_LOCATION(scope_node->parser, message_loc, call_node->base.node_id);
LINK_ELEMENT *opt_new_prelude = LAST_ELEMENT(ret);
LABEL *else_label = NEW_LABEL(location.line);
LABEL *end_label = NEW_LABEL(location.line);
LABEL *retry_end_l = NEW_LABEL(location.line);
@ -3661,6 +3659,8 @@ pm_compile_call(rb_iseq_t *iseq, const pm_call_node_t *call_node, LINK_ANCHOR *c
add_trace_branch_coverage(iseq, ret, &code_location, node_id, 0, "then", branches);
}
LINK_ELEMENT *opt_new_prelude = LAST_ELEMENT(ret);
int flags = 0;
struct rb_callinfo_kwarg *kw_arg = NULL;

5
test/ruby/test_optimization.rb
View File

@ -1216,6 +1216,11 @@ class TestRubyOptimization < Test::Unit::TestCase
RUBY
end
def test_opt_new_with_safe_navigation
payload = nil
assert_nil payload&.new
end
def test_opt_new
pos_initialize = "
def initialize a, b