Project

General

Profile

Actions

Bug #8719

closed

r42096 make bm_app_factorial.rb slow

Added by authorNari (Narihiro Nakamura) over 10 years ago. Updated over 10 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 2.1.0dev (2013-07-21 trunk 42096) [x86_64-linux]
[ruby-dev:47564]

Description

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

Updated by authorNari (Narihiro Nakamura) over 10 years ago

  • Description updated (diff)
Actions #2

Updated by akr (Akira Tanaka) over 10 years ago

  • 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.

Updated by akr (Akira Tanaka) over 10 years ago

2013/8/2 authorNari (Narihiro Nakamura) :

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]

Updated by authorNari (Narihiro Nakamura) over 10 years ago

2013年8月2日 12:30 Tanaka Akira :

2013/8/2 authorNari (Narihiro Nakamura) :

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)

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0