Bug #8719

r42096 make bm_app_factorial.rb slow

Added by Narihiro Nakamura 9 months ago. Updated 9 months ago.

[ruby-dev:47564]
Status:Closed
Priority:Normal
Assignee:Akira Tanaka
Category:core
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

Description

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

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

% time ./miniruby -v benchmark/bmappfactorial.rb
ruby 2.1.0dev (2013-07-21 trunk 42096) [x8664-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 9 months ago

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

History

#1 Updated by Narihiro Nakamura 9 months ago

  • Description updated (diff)

#2 Updated by Akira Tanaka 9 months 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 (barymul): Swap x and y for barymul1 if x is longer than y. [Bug #8719] Reported by Narihiro Nakamura.

#3 Updated by Akira Tanaka 9 months ago

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

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

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

おっと、x と y で長いほうを後にする処理が抜けてました。
r42324 で対処したつもりです。
--
[田中 哲][たなか あきら][Tanaka Akira]

#4 Updated by Narihiro Nakamura 9 months ago

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

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

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

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

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

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

--
Narihiro Nakamura (nari)

Also available in: Atom PDF