Remove assertion in encoded_iseq_trace_instrument (#15616)

luke-gruber · web-flow · commit 74bfb1606d09 · 2025-12-18T13:51:33.000-05:00
`encoded_iseq_trace_instrument` is safe to call in a ractor if the iseq is new. In that case, the VM lock is not taken. This assertion was added in 4fb537b.
diff --git a/iseq.c b/iseq.c
@@ -3936,7 +3936,6 @@ rb_vm_insn_decode(const VALUE encoded)
 static inline int
 encoded_iseq_trace_instrument(VALUE *iseq_encoded_insn, rb_event_flag_t turnon, bool remain_traced)
 {
-    ASSERT_vm_locking();
     st_data_t key = (st_data_t)*iseq_encoded_insn;
     st_data_t val;
 

Original file line number	Diff line number	Diff line change
`@@ -3936,7 +3936,6 @@ rb_vm_insn_decode(const VALUE encoded)`
`3936`	`3936`	`static inline int`
`3937`	`3937`	`encoded_iseq_trace_instrument(VALUE *iseq_encoded_insn, rb_event_flag_t turnon, bool remain_traced)`
`3938`	`3938`	`{`
`3939`		`- ASSERT_vm_locking();`
`3940`	`3939`	`st_data_t key = (st_data_t)*iseq_encoded_insn;`
`3941`	`3940`	`st_data_t val;`
`3942`	`3941`