util: remove unreachable defensive coding

Now that we are using primordials in the first part of isIdenticalTypedArrayType(), the defensive coding to get the correct result (when Symbol.toStringTag is manipulated) is no longer reachable or necessary. Remove the code. Refs: https://coverage.nodejs.org/coverage-873d21cdc1266273/lib/internal/util/comparisons.js.html#L135 PR-URL: #36744 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

Author: Trott

lib/internal/util/comparisons.js

@@ -44,15 +44,6 @@ const{
  isSymbolObject,
  isFloat32Array,
  isFloat64Array,
- isUint8Array,
- isUint8ClampedArray,
- isUint16Array,
- isUint32Array,
- isInt8Array,
- isInt16Array,
- isInt32Array,
- isBigInt64Array,
- isBigUint64Array
 }=types;
 const{
  getOwnNonIndexProperties,
@@ -126,38 +117,6 @@ function isEqualBoxedPrimitive(val1, val2){
 assert.fail(`Unknown boxed type ${val1}`);
 }
 
-functionisIdenticalTypedArrayType(a,b){
-// Fast path to reduce type checks in the common case.
-constcheck=types[`is${TypedArrayPrototypeGetSymbolToStringTag(a)}`];
-if(check!==undefined&&check(a)){
-returncheck(b);
-}
-// Manipulated Symbol.toStringTag.
-for(constcheckof[
-isUint16Array,
-isUint32Array,
-isInt8Array,
-isInt16Array,
-isInt32Array,
-isFloat32Array,
-isFloat64Array,
-isBigInt64Array,
-isBigUint64Array,
-isUint8ClampedArray,
-isUint8Array
-]){
-if(check(a)){
-returncheck(b);
-}
-}
-/* c8 ignore next 4 */
-assert.fail(
-'Unknown TypedArray type checking '+
-`${TypedArrayPrototypeGetSymbolToStringTag(a)}${a}\n`+
-`and ${TypedArrayPrototypeGetSymbolToStringTag(b)}${b}`
-);
-}
-
 // Notes: Type tags are historical [[Class]] properties that can be set by
 // FunctionTemplate::SetClassName() in C++ or Symbol.toStringTag in JS
 // and retrieved using Object.prototype.toString.call(obj) in JS
@@ -241,8 +200,10 @@ function innerDeepEqual(val1, val2, strict, memos){
 returnfalse;
 }
 }elseif(isArrayBufferView(val1)){
-if(!isIdenticalTypedArrayType(val1,val2))
+if(TypedArrayPrototypeGetSymbolToStringTag(val1)!==
+TypedArrayPrototypeGetSymbolToStringTag(val2)){
 returnfalse;
+}
 if(!strict&&(isFloat32Array(val1)||isFloat64Array(val1))){
 if(!areSimilarFloatArrays(val1,val2)){
 returnfalse;