summaryrefslogtreecommitdiff
path: root/include/ruby/debug.h
diff options
context:
space:
mode:
authorNARUSE, Yui <naruse@airemix.jp>2023-01-18 18:56:51 +0900
committerNARUSE, Yui <naruse@airemix.jp>2023-01-18 18:56:51 +0900
commit1fb5eb5740d4c4f1fc34a4a50bc0482eac27b545 (patch)
tree54be1f45b0e775d0478271d9ed492333db998312 /include/ruby/debug.h
parentf7e9b79f815e08e0a29fff11f77466f4ffb92520 (diff)
merge revision(s) aeddc19340c7116d48fac3080553fbb823857d16: [Backport #19316]
YJIT: Save PC and SP before calling leaf builtins (#7090) Previously, we did not update `cfp->sp` before calling the C function of ISEQs marked with `Primitive.attr! "inline"` (leaf builtins). This caused the GC to miss temporary values on the stack in case the function allocates and triggers a GC run. Right now, there is only a few leaf builtins in numeric.rb on Integer methods such as `Integer#~`. Since these methods only allocate when operating on big numbers, we missed this issue. Fix by saving PC and SP before calling the functions -- our usual protocol for calling C functions that may allocate on the GC heap. [Bug #19316] --- test/ruby/test_yjit.rb | 16 ++++++++++++++++ yjit/src/codegen.rs | 4 ++++ 2 files changed, 20 insertions(+)
Diffstat (limited to 'include/ruby/debug.h')
0 files changed, 0 insertions, 0 deletions