Bug #8719: r42096 make bm_app_factorial.rb slow

• Status: Closed
• Assignee: akr (Akira Tanaka)
• Target version: 2.1.0
• ruby -v: ruby 2.1.0dev (2013-07-21 trunk 42096) [x86_64-linux]
• Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

[authorNari (Narihiro Nakamura)]

r42096 のコミットでbm_app_factorial.rbがかなり遅くなるようです。
しょうがないということでしたらrejectしてください。
とりあえずご報告までということで。

% time ./miniruby -v benchmark/bm_app_factorial.rb
ruby 2.1.0dev (2013-07-21 trunk 42095) [x86_64-linux]
./miniruby -v benchmark/bm_app_factorial.rb 1.17s user 0.00s system 99% cpu 1.184 total

% time ./miniruby -v benchmark/bm_app_factorial.rb
ruby 2.1.0dev (2013-07-21 trunk 42096) [x86_64-linux]
./miniruby -v benchmark/bm_app_factorial.rb 2.11s user 0.02s system 99% cpu 2.137 total

[authorNari (Narihiro Nakamura)]

• Description updated (diff)

[akr (Akira Tanaka)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r42324.
Narihiro, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• bignum.c (bary_mul): Swap x and y for bary_mul1 if x is longer than y.
  [ruby-dev:47565] [Bug #8719] Reported by Narihiro Nakamura.

[akr (Akira Tanaka)]

2013/8/2 authorNari (Narihiro Nakamura) [e-mail]:

Bug #8719: r42096 make bm_app_factorial.rb slow
https://bugs.ruby-lang.org/issues/8719#change-40811

r42096 のコミットでbm_app_factorial.rbがかなり遅くなるようです。
しょうがないということでしたらrejectしてください。
とりあえずご報告までということで。

おっと、x と y で長いほうを後にする処理が抜けてました。
r42324 で対処したつもりです。

[田中 哲][たなか あきら][Tanaka Akira]

[authorNari (Narihiro Nakamura)]

2013年8月2日 12:30 Tanaka Akira [e-mail]:

2013/8/2 authorNari (Narihiro Nakamura) [e-mail]:

Bug #8719: r42096 make bm_app_factorial.rb slow
https://bugs.ruby-lang.org/issues/8719#change-40811

r42096 のコミットでbm_app_factorial.rbがかなり遅くなるようです。
しょうがないということでしたらrejectしてください。
とりあえずご報告までということで。

おっと、x と y で長いほうを後にする処理が抜けてました。
r42324 で対処したつもりです。

手元で直っていることを確認しました。ありがとうございます！

--
Narihiro Nakamura (nari)