util: use faster -0 check

mscdex · mscdex · commit d545c948c238 · 2017-10-04T03:51:44.000-04:00
PR-URL: nodejs#15726 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Benedikt Meurer <benedikt.meurer@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
diff --git a/benchmark/util/inspect.js b/benchmark/util/inspect.js
@@ -22,7 +22,8 @@ const bench = common.createBenchmark(main, {
     'Error',
     'Array',
     'TypedArray',
-    'TypedArray_extra'
+    'TypedArray_extra',
+    'Number'
   ],
   option: Object.keys(opts)
 });
@@ -92,6 +93,9 @@ function main({ method, n, option }) {
       obj[Symbol('baz')] = 5;
       benchmark(n, obj, options);
       break;
+    case 'Number':
+      benchmark(n, 0, options);
+      break;
     default:
       throw new Error(`Unsupported method "${method}"`);
   }
diff --git a/lib/util.js b/lib/util.js
@@ -615,8 +615,10 @@ function formatValue(ctx, value, recurseTimes, ln) {
 }
 
 function formatNumber(fn, value) {
-  // Format -0 as '-0'. Strict equality won't distinguish 0 from -0.
-  if (Object.is(value, -0))
+  // Format -0 as '-0'. A `value === -0` check won't distinguish 0 from -0.
+  // Using a division check is currently faster than `Object.is(value, -0)`
+  // as of V8 6.1.
+  if (1 / value === -Infinity)
     return fn('-0', 'number');
   return fn(`${value}`, 'number');
 }

Original file line number	Diff line number	Diff line change
`@@ -615,8 +615,10 @@ function formatValue(ctx, value, recurseTimes, ln) {`
`615`	`615`	`}`
`616`	`616`
`617`	`617`	`function formatNumber(fn, value) {`
`618`		`- // Format -0 as '-0'. Strict equality won't distinguish 0 from -0.`
`619`		`- if (Object.is(value, -0))`
	`618`	+ // Format -0 as '-0'. A `value === -0` check won't distinguish 0 from -0.
	`619`	+ // Using a division check is currently faster than `Object.is(value, -0)`
	`620`	`+ // as of V8 6.1.`
	`621`	`+ if (1 / value === -Infinity)`
`620`	`622`	`return fn('-0', 'number');`
`621`	`623`	return fn(`${value}`, 'number');
`622`	`624`	`}`