* [ruby-dev:50983] [Ruby master Feature#13368] Improve performance of Array#sum with float elements [not found] <redmine.issue-13368.20170327035053.12374@ruby-lang.org> @ 2020-12-26 1:27 ` hsbt 0 siblings, 0 replies; only message in thread From: hsbt @ 2020-12-26 1:27 UTC (permalink / raw) To: ruby-dev Issue #13368 has been updated by hsbt (Hiroshi SHIBATA). Assignee set to mrkn (Kenta Murata) Status changed from Open to Closed https://github.com/ruby/ruby/pull/1555 was merged at https://github.com/ruby/ruby/commit/2d001003e4b3a6c20ead09ed54b6726a7669f457 ---------------------------------------- Feature #13368: Improve performance of Array#sum with float elements https://bugs.ruby-lang.org/issues/13368#change-89526 * Author: watson1978 (Shizuo Fujita) * Status: Closed * Priority: Normal * Assignee: mrkn (Kenta Murata) ---------------------------------------- The declaration of local variable in loop, it will initialize local variable for each run of the loop with clang generated code. So, it shouldn't declare the local variable in heavy loop. Array#sum with float elements will be faster around 30%. ### Before ~~~ user system total real 3.320000 0.010000 3.330000 ( 3.336088) ~~~ ### After ~~~ user system total real 2.590000 0.010000 2.600000 ( 2.602399) ~~~ ### Test code ~~~ require 'benchmark' Benchmark.bmbm do |x| ary = [] 10000.times { ary << Random.rand } x.report do 50000.times do ary.sum end end end ~~~ ### Patch https://github.com/ruby/ruby/pull/1555 -- https://bugs.ruby-lang.org/ ^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-12-26 1:27 UTC | newest] Thread overview: (only message) (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <redmine.issue-13368.20170327035053.12374@ruby-lang.org> 2020-12-26 1:27 ` [ruby-dev:50983] [Ruby master Feature#13368] Improve performance of Array#sum with float elements hsbt
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).