assert,util: improve unequal number comparison performance

This improves the performance to compare unequal numbers while doing
a deep equal comparison. Comparing for NaN is faster by checking
`variable !== variable` than by using `Number.isNaN()`.

PR-URL: #57619
Reviewed-By: Jordan Harband <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>

Author: BridgeAR

lib/internal/util/comparisons.js

@@ -8,7 +8,6 @@ const {
   BooleanPrototypeValueOf,
   DatePrototypeGetTime,
   Error,
-  NumberIsNaN,
   NumberPrototypeValueOf,
   ObjectGetOwnPropertySymbols: getOwnSymbols,
   ObjectGetPrototypeOf,
@@ -185,7 +184,9 @@ function innerDeepEqual(val1, val2, mode, memos) {
   // Check more closely if val1 and val2 are equal.
   if (mode !== kLoose) {
     if (typeof val1 === 'number') {
-      return NumberIsNaN(val1) && NumberIsNaN(val2);
+      // Check for NaN
+      // eslint-disable-next-line no-self-compare
+      return val1 !== val1 && val2 !== val2;
     }
     if (typeof val2 !== 'object' ||
         typeof val1 !== 'object' ||
@@ -197,8 +198,9 @@ function innerDeepEqual(val1, val2, mode, memos) {
   } else {
     if (val1 === null || typeof val1 !== 'object') {
       return (val2 === null || typeof val2 !== 'object') &&
-             // eslint-disable-next-line eqeqeq
-             (val1 == val2 || (NumberIsNaN(val1) && NumberIsNaN(val2)));
+             // Check for NaN
+             // eslint-disable-next-line eqeqeq, no-self-compare
+             (val1 == val2 || (val1 !== val1 && val2 !== val2));
     }
     if (val2 === null || typeof val2 !== 'object') {
       return false;
@@ -501,7 +503,9 @@ function findLooseMatchingPrimitives(prim) {
       // a regular number and not NaN.
       // Fall through
     case 'number':
-      if (NumberIsNaN(prim)) {
+      // Check for NaN
+      // eslint-disable-next-line no-self-compare
+      if (prim !== prim) {
         return false;
       }
   }