[pull] master from ruby:master - #1261
Merged
Merged
Conversation
bundler.io has been wound down and its content now lives on RubyGems Guides: the Bundler man pages under /command-reference/, the Gemfile reference at /gemfile/, and the remaining guides as individual pages. The redirects still work, but linking to the real locations keeps `gem help gem_dependencies`, `bundle issue`, and the man page cross references from depending on them. Two references stay on bundler.io. The homepage in lib/bundler/source/metadata.rb has to match the published bundler gemspec, and the team pages linked from doc/POLICIES.md have no replacement yet. ruby/rubygems@69aa22cd41 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
microsoft/vcpkg bumped `scripts/vcpkg-tools.json` to schema version 2, which the vcpkg.exe bundled in the runner images cannot parse, so every Windows job started failing at `vcpkg install` right after the `git pull`. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
We don't need to call obj_free on certain types of objects anymore including
T_FLOAT, T_RATIONAL, T_COMPLEX, and T_OBJECT. It makes freeing those objects
faster.
Benchmark:
i = 0
while i < 10_000_000
Object.new
i += 1
end
Before:
Time (mean ± σ): 1.435 s ± 0.013 s [User: 1.706 s, System: 1.970 s]
Range (min … max): 1.418 s … 1.461 s 10 runs
After:
Time (mean ± σ): 1.230 s ± 0.027 s [User: 1.335 s, System: 1.743 s]
Range (min … max): 1.206 s … 1.294 s 10 runs
ruby/mmtk@52f01ba90f
The "does not timeout on massive files" integration test in spec/syntax_suggest/integration/syntax_suggest_spec.rb is only activated when SYNTAX_SUGGEST_TIMEOUT is set. The CI workflow sets it to 10 for ppc64le/s390x runners, but this is insufficient: on ppc64le, processing the 9,234-line syntax_tree.rb.txt fixture takes more than 10 seconds, causing the test to fail. Increase SYNTAX_SUGGEST_TIMEOUT to 60 seconds (consistent with the 10x scale factor applied to RUBY_TEST_TIMEOUT_SCALE for these slower architectures) to give the algorithm enough time to complete. Fixes: make-ibm (check, ubuntu-24.04-ppc64le) CI failure
* ZJIT: Profile caller splat length distributions Related to: Shopify#1004 Before specializing sends with caller splats, collect the distribution of caller splat array lengths. The splat length determines how many `ArrayAref` operations and positional arguments a `SendDirect` path needs. Specializing for only one observed length could cause repeated side exits when a call site sees varying lengths, so we need to understand whether these profiles are monomorphic or polymorphic first. This does not change caller splat specialization or fallback behavior yet. * ZJIT: Refine caller splat length profiling Store splat length distributions in a side map since only splat call sites use them. Record unknown lengths instead of dropping those samples so the length profile stays aligned with the operand type profile. Co-authored-by: Max Bernstein <tekknolagi@gmail.com>
Reduce the number of branches and only look up the shape once instead of twice. Brings a profiling-heavy run (profiling for 1000 calls) of lobsters from 700ms to 660ms profile time. Unfortunately getting the current time for stats is still 50% of all profile time.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )