lukechilds/node - node - Gitea: Git with a cup of tea

17544 Commits

131 Branches

446 Tags

325 MiB

Tree: 594b5d7b89

Commit Graph

Author	SHA1	Message	Date
Bartosz Sosnowski	ea61ce518b	benchmark: remove forced optimization from tls This removes all instances of %OptimizeFunctionOnNextCall from tls benchmarks PR-URL: https://github.com/nodejs/node/pull/9615 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>	8 years ago
James M Snell	f3d5efa3ee	tls: avoid calling Buffer.byteLength multiple times There's no reason to be calling Buffer.byteLength() twice. Small perf improvement Run 1: tls/convertprotocols.js n=1 v6.2.1 = 11852, new = 12204 ...... -2.89% tls/convertprotocols.js n=50000 v6.2.1 = 515660, new = 570610 ..... -9.63% Run 2: tls/convertprotocols.js n=1 v6.2.1 = 11729, new = 12045 ...... -2.62% tls/convertprotocols.js n=50000 v6.2.1 = 512080, new = 637730 ..... -19.70% PR-URL: https://github.com/nodejs/node/pull/7236 Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>	9 years ago

Author

SHA1

Message

Date

Bartosz Sosnowski

ea61ce518b

benchmark: remove forced optimization from tls

This removes all instances of %OptimizeFunctionOnNextCall from tls
benchmarks

PR-URL: https://github.com/nodejs/node/pull/9615
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

8 years ago

James M Snell

f3d5efa3ee

tls: avoid calling Buffer.byteLength multiple times

There's no reason to be calling Buffer.byteLength() twice.
Small perf improvement

Run 1:
tls/convertprotocols.js n=1     v6.2.1 = 11852,  new = 12204 ......  -2.89%
tls/convertprotocols.js n=50000 v6.2.1 = 515660, new = 570610 .....  -9.63%

Run 2:
tls/convertprotocols.js n=1     v6.2.1 = 11729,  new = 12045 ......  -2.62%
tls/convertprotocols.js n=50000 v6.2.1 = 512080, new = 637730 ..... -19.70%

PR-URL: https://github.com/nodejs/node/pull/7236
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>

9 years ago

2 Commits (594b5d7b8970a68d9cd66f629cf1862cfca68412)