Project

General

Profile

Bug #8719

r42096 make bm_app_factorial.rb slow

Added by Narihiro Nakamura almost 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
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

Associated revisions

Revision 42324
Added by Akira Tanaka almost 3 years ago

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

Revision 42324
Added by Akira Tanaka almost 3 years ago

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

Revision 42324
Added by Akira Tanaka almost 3 years ago

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

History

#1 [ruby-dev:47565] Updated by Narihiro Nakamura almost 3 years ago

  • Description updated (diff)

#2 Updated by Akira Tanaka almost 3 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. [Bug #8719] Reported by Narihiro Nakamura.

#3 [ruby-dev:47566] Updated by Akira Tanaka almost 3 years ago

2013/8/2 authorNari (Narihiro Nakamura) authorNari@gmail.com:

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]

#4 [ruby-dev:47567] Updated by Narihiro Nakamura almost 3 years ago

2013年8月2日 12:30 Tanaka Akira akr@fsij.org:

2013/8/2 authorNari (Narihiro Nakamura) authorNari@gmail.com:

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)

Also available in: Atom PDF