mirror of
https://github.com/ruby/ruby.git
synced 2025-09-18 01:54:00 +02:00
1fb5eb5740
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(+) |
||
|---|---|---|
| .. | ||
| asm | ||
| backend | ||
| codegen.rs | ||
| core.rs | ||
| cruby.rs | ||
| cruby_bindings.inc.rs | ||
| disasm.rs | ||
| invariants.rs | ||
| lib.rs | ||
| options.rs | ||
| stats.rs | ||
| utils.rs | ||
| virtualmem.rs | ||
| yjit.rs | ||