diff options
author | John Hawthorn <[email protected]> | 2025-04-30 14:17:48 -0700 |
---|---|---|
committer | John Hawthorn <[email protected]> | 2025-04-30 15:36:58 -0700 |
commit | 7866e124a852c344b5762eb917c03a1f95d9058d (patch) | |
tree | e1bc5e82841bb838e76bae46e948eeb9c5f930b7 /test/prism/version_test.rb | |
parent | 73fdd90315a7038b060ec78ca0984951d13ccb4f (diff) |
When doing a coroutine transfer from one thread to another, there's a
risk that the compiler will reuse an address from TLS before the
transfer to the new thread.
These VM assertions are all in places we would not otherwise be reading
from TLS, but using the value of `ec` or `cr` passed in. Switching these
to test against rb_current_ec_noinline() instead ensures there isn't an
optimization applied to how we read ruby_current_ec.
Currently it seems we were hitting this on LLVM 18 specifically, but I
don't know of any reason other versions wouldn't have the same issue.
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/13217
Diffstat (limited to 'test/prism/version_test.rb')
0 files changed, 0 insertions, 0 deletions