Title: [205361] trunk/Source/_javascript_Core
Revision
205361
Author
[email protected]
Date
2016-09-02 12:17:27 -0700 (Fri, 02 Sep 2016)

Log Message

GetByValWithThis: fix opInfo in DFG creation
https://bugs.webkit.org/show_bug.cgi?id=161541

Patch by JF Bastien <[email protected]> on 2016-09-02
Reviewed by Saam Barati.

super-get-by-val-with-this-monomorphic might be 1.0148x faster after this change.

* dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::parseBlock): fix OpInfo

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (205360 => 205361)


--- trunk/Source/_javascript_Core/ChangeLog	2016-09-02 19:14:46 UTC (rev 205360)
+++ trunk/Source/_javascript_Core/ChangeLog	2016-09-02 19:17:27 UTC (rev 205361)
@@ -1,3 +1,15 @@
+2016-09-02  JF Bastien  <[email protected]>
+
+        GetByValWithThis: fix opInfo in DFG creation
+        https://bugs.webkit.org/show_bug.cgi?id=161541
+
+        Reviewed by Saam Barati.
+
+        super-get-by-val-with-this-monomorphic might be 1.0148x faster after this change.
+
+        * dfg/DFGByteCodeParser.cpp:
+        (JSC::DFG::ByteCodeParser::parseBlock): fix OpInfo
+
 2016-09-02  Chris Dumez  <[email protected]>
 
         Object.preventExtensions() should throw cross-origin

Modified: trunk/Source/_javascript_Core/dfg/DFGByteCodeParser.cpp (205360 => 205361)


--- trunk/Source/_javascript_Core/dfg/DFGByteCodeParser.cpp	2016-09-02 19:14:46 UTC (rev 205360)
+++ trunk/Source/_javascript_Core/dfg/DFGByteCodeParser.cpp	2016-09-02 19:17:27 UTC (rev 205361)
@@ -4134,7 +4134,7 @@
             Node* base = get(VirtualRegister(currentInstruction[2].u.operand));
             Node* thisValue = get(VirtualRegister(currentInstruction[3].u.operand));
             Node* property = get(VirtualRegister(currentInstruction[4].u.operand));
-            Node* getByValWithThis = addToGraph(GetByValWithThis, OpInfo(prediction), base, thisValue, property);
+            Node* getByValWithThis = addToGraph(GetByValWithThis, OpInfo(), OpInfo(prediction), base, thisValue, property);
             set(VirtualRegister(currentInstruction[1].u.operand), getByValWithThis);
 
             NEXT_OPCODE(op_get_by_val_with_this);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to