* [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).