diff options
author | Victor Felder <victor@draft.li> | 2017-01-02 22:28:04 +0100 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2017-01-05 12:33:10 -0800 |
commit | e52fee50a08c9d9ca6215149cc8e8c0b174e32dd (patch) | |
tree | bd2224ebad231d074e75c06553b895d22acc43b4 /benchmark/run.js | |
parent | 26b8c2127dfbf1610b4f210a2d3ee138ecb187b0 (diff) | |
download | android-node-v8-e52fee50a08c9d9ca6215149cc8e8c0b174e32dd.tar.gz android-node-v8-e52fee50a08c9d9ca6215149cc8e8c0b174e32dd.tar.bz2 android-node-v8-e52fee50a08c9d9ca6215149cc8e8c0b174e32dd.zip |
events: avoid emit() eager deopt
This commit makes sure EventEmitter.emit() doesn't get deoptimized by
V8. The deopt happens when accessing out of bound indexes of the
`arguments` object.
This issue has been raised here: #10323 and this specific case might
become a more serious performance issue in upcoming V8 releases.
PR-URL: https://github.com/nodejs/node/pull/10568
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Diffstat (limited to 'benchmark/run.js')
0 files changed, 0 insertions, 0 deletions