Skip to content

Forward port from GH-13115 from ruby/ruby#27

Merged
eightbitraptor merged 2 commits into
mainfrom
forward-port-from-ruby-master
Apr 30, 2025
Merged

Forward port from GH-13115 from ruby/ruby#27
eightbitraptor merged 2 commits into
mainfrom
forward-port-from-ruby-master

Conversation

@hsbt
Copy link
Copy Markdown
Member

@hsbt hsbt commented Apr 15, 2025

Copy link
Copy Markdown
Member

@peterzhu2118 peterzhu2118 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

byroot and others added 2 commits April 30, 2025 14:41
This inverse table is only useful if `ObjectSpace._id2ref` is used,
which is extremely rare. The only notable exception is the `drb` gem
and even then it has an option not to rely on `_id2ref`.

So if we assume this table will never be looked up, we can just
not maintain it, and if it turns out `_id2ref` is called, we
can lock the VM and re-build it.

```
compare-ruby: ruby 3.5.0dev (2025-04-10T09:44:40Z master 684cfa42d7) +YJIT +PRISM [arm64-darwin24]
built-ruby: ruby 3.5.0dev (2025-04-10T10:13:43Z lazy-id-to-obj d3aa9626cc) +YJIT +PRISM [arm64-darwin24]
warming up..

```
Bumps [crossbeam-channel](https://github.com/crossbeam-rs/crossbeam) from 0.5.13 to 0.5.15.
- [Release notes](https://github.com/crossbeam-rs/crossbeam/releases)
- [Changelog](https://github.com/crossbeam-rs/crossbeam/blob/master/CHANGELOG.md)
- [Commits](crossbeam-rs/crossbeam@crossbeam-channel-0.5.13...crossbeam-channel-0.5.15)

---
updated-dependencies:
- dependency-name: crossbeam-channel
  dependency-version: 0.5.15
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@eightbitraptor eightbitraptor force-pushed the forward-port-from-ruby-master branch from fdae5b4 to a2eaa7f Compare April 30, 2025 13:42
@eightbitraptor eightbitraptor merged commit 36c6bbc into main Apr 30, 2025
20 checks passed
@eightbitraptor eightbitraptor deleted the forward-port-from-ruby-master branch April 30, 2025 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants