Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add an additional test to module_eqq #13308

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

Merged
merged 1 commit into from
May 13, 2025

Conversation

jhawthorn
Copy link
Member

@jhawthorn jhawthorn commented May 13, 2025

Another case to help demonstrate the namespace performance penalty

❯ make benchmark ITEM=module_eqq
/home/jhawthorn/.rubies/ruby-3.4.1/bin/ruby --disable=gems -rrubygems -I./benchmark/lib ./benchmark/benchmark-driver/exe/benchmark-driver \
            --executables="compare-ruby::/home/jhawthorn/.rubies/ruby-3.4.1/bin/ruby --disable=gems -I.ext/common --disable-gem" \
            --executables="built-ruby::./miniruby -I./lib -I. -I.ext/common  ./tool/runruby.rb --extout=.ext  -- --disable-gems --disable-gem" \
            --output=markdown --output-compare -v $(find ./benchmark -maxdepth 1 -name 'module_eqq' -o -name '*module_eqq*.yml' -o -name '*module_eqq*.rb' | sort)
compare-ruby: ruby 3.4.1 (2024-12-25 revision 48d4efcb85) +PRISM [x86_64-linux]
built-ruby: ruby 3.5.0dev (2025-05-12T07:49:59Z master 131ba059ca) +PRISM [x86_64-linux]
# Iteration per second (i/s)
|                             |compare-ruby|built-ruby|
|:----------------------------|-----------:|---------:|
|simple_class_eqq_simple_obj  |     62.318M|   48.763M|
|                             |       1.28x|         -|
|medium_class_eqq_simple_obj  |     62.044M|   49.489M|
|                             |       1.25x|         -|
|simple_class_eqq_medium_obj  |     61.798M|   50.382M|
|                             |       1.23x|         -|
|simple_class_eqq_large_obj   |     61.179M|   49.809M|
|                             |       1.23x|         -|
|simple_class_eqq_huge_obj    |     62.527M|   50.187M|
|                             |       1.25x|         -|
|simple_class_eqq_module      |     63.095M|   50.237M|
|                             |       1.26x|         -|
|module_eqq_module            |      3.261M|    1.952M|
|                             |       1.67x|         -|

@jhawthorn jhawthorn merged commit 9b8c846 into ruby:master May 13, 2025
82 checks passed
@jhawthorn jhawthorn deleted the module_eqq_another_test branch May 13, 2025 02:05
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.

1 participant