Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DOC] Add production performance tips to yjit.md #8472

Merged
merged 4 commits into from Sep 18, 2023
Merged

Conversation

k0kubun
Copy link
Member

@k0kubun k0kubun commented Sep 18, 2023

  • Added "Speeding Up YJIT" and "Saving YJIT Memory Usage" that contain tips for tuning YJIT in production.
  • Tweaked a couple of places in "Performance Tips" since exceptions and Integer#times are sped up in Ruby 3.3.

@matzbot matzbot requested a review from a team September 18, 2023 21:52
doc/yjit/yjit.md Outdated Show resolved Hide resolved
doc/yjit/yjit.md Outdated Show resolved Hide resolved
doc/yjit/yjit.md Outdated Show resolved Hide resolved
Copy link
Contributor

@maximecb maximecb 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 for writing this Kokubun 🙏

Wrote some small suggestions.

k0kubun and others added 2 commits September 18, 2023 15:14
Co-authored-by: Maxime Chevalier-Boisvert <maxime.chevalierboisvert@shopify.com>
Copy link
Member

@ioquatix ioquatix left a comment

Choose a reason for hiding this comment

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

LGTM!

@k0kubun k0kubun merged commit 0a2c4a6 into ruby:master Sep 18, 2023
4 checks passed
@k0kubun k0kubun deleted the yjit-doc branch September 18, 2023 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants