Changeset 245409 in webkit

Timestamp:

May 16, 2019, 3:47:52 PM (7 years ago)

Author:

Kocsen Chung

Message:

Revert "Apply patch. rdar://problem/50754976"

Location:

branches/safari-607.2.1.2-branch/Source/JavaScriptCore/bytecompiler

Files:

• BytecodeGenerator.cpp (modified) (2 diffs)
• BytecodeGenerator.h (modified) (1 diff)

branches/safari-607.2.1.2-branch/Source/JavaScriptCore/bytecompiler/BytecodeGenerator.cpp

@@ -1709 +1709 @@
 }
 
-bool BytecodeGenerator::emitEqualityOpImpl(RegisterID* dst, RegisterID* src1, RegisterID* src2)
+template<typename EqOp>
+RegisterID* BytecodeGenerator::emitEqualityOp(RegisterID* dst, RegisterID* src1, RegisterID* src2)
 {
     if (!canDoPeepholeOptimization())
@@ -1724 +1725 @@
                 rewind();
                 OpIsUndefined::emit(this, dst, op.m_value);
-                return true;
+                return dst;
             }
             if (value == "boolean") {
                 rewind();
                 OpIsBoolean::emit(this, dst, op.m_value);
-                return true;
+                return dst;
             }
             if (value == "number") {
                 rewind();
                 OpIsNumber::emit(this, dst, op.m_value);
-                return true;
+                return dst;
             }
             if (value == "string") {
                 rewind();
                 OpIsCellWithType::emit(this, dst, op.m_value, StringType);
-                return true;
+                return dst;
             }
             if (value == "symbol") {
                 rewind();
                 OpIsCellWithType::emit(this, dst, op.m_value, SymbolType);
-                return true;
+                return dst;
             }
             if (Options::useBigInt() && value == "bigint") {
                 rewind();
                 OpIsCellWithType::emit(this, dst, op.m_value, BigIntType);
-                return true;
+                return dst;
             }
             if (value == "object") {
                 rewind();
                 OpIsObjectOrNull::emit(this, dst, op.m_value);
-                return true;
+                return dst;
             }
             if (value == "function") {
                 rewind();
                 OpIsFunction::emit(this, dst, op.m_value);
-                return true;
+                return dst;
             }
         }
     }
 
-    return false;
+    EqOp::emit(this, dst, src1, src2);
+    return dst;
 }
 

branches/safari-607.2.1.2-branch/Source/JavaScriptCore/bytecompiler/BytecodeGenerator.h

@@ -712 +712 @@
 
         template<typename EqOp>
-        RegisterID* emitEqualityOp(RegisterID* dst, RegisterID* src1, RegisterID* src2)
-        {
-            if (!emitEqualityOpImpl(dst, src1, src2))
-                EqOp::emit(this, dst, src1, src2);
-            return dst;
-        }
-
-        bool emitEqualityOpImpl(RegisterID* dst, RegisterID* src1, RegisterID* src2);
-
+        RegisterID* emitEqualityOp(RegisterID* dst, RegisterID* src1, RegisterID* src2);
         RegisterID* emitCreateThis(RegisterID* dst);
         void emitTDZCheck(RegisterID* target);