ruby-changes:55316
From: k0kubun <ko1@a...>
Date: Sat, 13 Apr 2019 15:55:42 +0900 (JST)
Subject: [ruby-changes:55316] k0kubun:r67523 (trunk): Revert "GET_CFP and ec->cfp are different"
k0kubun 2019-04-13 15:55:35 +0900 (Sat, 13 Apr 2019) New Revision: 67523 https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=67523 Log: Revert "GET_CFP and ec->cfp are different" This reverts commit 30f71f4768e0babc60f5be6ca235d2c40268bc27. I've also overlooked we're doing RESTORE_REGS()... Modified files: trunk/vm_insnhelper.c Index: vm_insnhelper.c =================================================================== --- vm_insnhelper.c (revision 67522) +++ vm_insnhelper.c (revision 67523) @@ -3636,12 +3636,12 @@ vm_sendish( https://github.com/ruby/ruby/blob/trunk/vm_insnhelper.c#L3636 frame may have stack values in the local variables and the cancelling the caller frame will purge them. But directly calling mjit_exec is faster... */ - if (ec->cfp->iseq->body->catch_except_p) { - VM_ENV_FLAGS_SET(ec->cfp->ep, VM_FRAME_FLAG_FINISH); + if (GET_ISEQ()->body->catch_except_p) { + VM_ENV_FLAGS_SET(GET_EP(), VM_FRAME_FLAG_FINISH); return vm_exec(ec, true); } else if ((val = mjit_exec(ec)) == Qundef) { - VM_ENV_FLAGS_SET(ec->cfp->ep, VM_FRAME_FLAG_FINISH); + VM_ENV_FLAGS_SET(GET_EP(), VM_FRAME_FLAG_FINISH); return vm_exec(ec, false); } else { -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/