diff options
| author | Takashi Kokubun <takashikkbn@gmail.com> | 2026-03-27 14:28:01 -0700 |
|---|---|---|
| committer | Takashi Kokubun <takashikkbn@gmail.com> | 2026-03-27 16:26:12 -0700 |
| commit | 226f37059ec5f3ea3a1417e0bab630c64dbc8ac3 (patch) | |
| tree | b76b9b7e2116596eef5de51ce234cb8f811b7a87 /zjit | |
| parent | 4e5b0d3132c39b690ac50d7a49e96a95b8271067 (diff) | |
Rename cfp->iseq to cfp->_iseq and rb_zjit_cfp_iseq to rb_cfp_iseq
Per tenderlove's review, every cfp->iseq read should go through an
accessor that accounts for JITFrame. Rename the field to _iseq so
direct access causes a compile error, and rename the accessors to
drop the zjit prefix (rb_cfp_iseq, rb_cfp_has_iseq, rb_cfp_pc,
rb_cfp_has_pc).
Diffstat (limited to 'zjit')
| -rw-r--r-- | zjit/src/codegen.rs | 2 | ||||
| -rw-r--r-- | zjit/src/cruby_bindings.inc.rs | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/zjit/src/codegen.rs b/zjit/src/codegen.rs index c59dfbe525..9846a0e70a 100644 --- a/zjit/src/codegen.rs +++ b/zjit/src/codegen.rs @@ -3072,7 +3072,7 @@ c_callable! { // Clear jit_return so the interpreter reads cfp->pc and cfp->iseq directly. let pc = unsafe { rb_iseq_pc_at_idx(iseq, entry_insn_idxs[iseq_call.jit_entry_idx.to_usize()]) }; unsafe { rb_set_cfp_pc(cfp, pc) }; - unsafe { (*cfp).iseq = iseq }; + unsafe { (*cfp)._iseq = iseq }; unsafe { (*cfp).jit_return = std::ptr::null_mut() }; } diff --git a/zjit/src/cruby_bindings.inc.rs b/zjit/src/cruby_bindings.inc.rs index 9a82d6a29f..d8fdcbb845 100644 --- a/zjit/src/cruby_bindings.inc.rs +++ b/zjit/src/cruby_bindings.inc.rs @@ -1303,7 +1303,7 @@ pub struct rb_block__bindgen_ty_1 { pub struct rb_control_frame_struct { pub pc: *const VALUE, pub sp: *mut VALUE, - pub iseq: *const rb_iseq_t, + pub _iseq: *const rb_iseq_t, pub self_: VALUE, pub ep: *const VALUE, pub block_code: *const ::std::os::raw::c_void, |