Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Unroll simple loop abstractions #1253

Merged
merged 2 commits into from
Jul 29, 2016
Merged

Unroll simple loop abstractions #1253

merged 2 commits into from
Jul 29, 2016

Conversation

megawac
Copy link
Collaborator

@megawac megawac commented Jul 23, 2016

Figure there is no reason to not write these simple loops our selves.

Latest tag is  v2.0.1
Comparing v2.0.1 with current on Node v6.2.2
--------------------------------------
queue(1000) v2.0.1 x 345 ops/sec ±0.80% (30 runs sampled), 2.90ms per run
queue(1000) current x 352 ops/sec ±0.90% (30 runs sampled), 2.84ms per run
current is faster
--------------------------------------
queue(30000) v2.0.1 x 9.39 ops/sec ±3.62% (18 runs sampled), 106ms per run
queue(30000) current x 10.52 ops/sec ±2.89% (20 runs sampled), 95.1ms per run
current is faster
--------------------------------------
queue(100000) v2.0.1 x 2.53 ops/sec ±11.46% (6 runs sampled), 396ms per run
queue(100000) current x 3.05 ops/sec ±8.64% (6 runs sampled), 328ms per run
current is faster
--------------------------------------
queue(200000) v2.0.1 x 1.26 ops/sec ±16.98% (3 runs sampled), 797ms per run
queue(200000) current x 1.43 ops/sec ±11.58% (3 runs sampled), 698ms per run
current is faster
--------------------------------------
current faster overall (1120ms total vs. 1300ms total)
current won more benchmarks (4 vs. 0)

@megawac megawac merged commit 60dc3b6 into master Jul 29, 2016
@megawac megawac deleted the loops branch July 29, 2016 18:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant