Title: [207765] trunk/Source/_javascript_Core
Revision
207765
Author
mark....@apple.com
Date
2016-10-24 10:53:40 -0700 (Mon, 24 Oct 2016)

Log Message

Dumping of op_negate bytecode is broken.
https://bugs.webkit.org/show_bug.cgi?id=163896

Reviewed by Darin Adler.

It thinks the bytecode consists of only 3 machine words, when it consists of 4.
This is now fixed.

* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::dumpBytecode):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (207764 => 207765)


--- trunk/Source/_javascript_Core/ChangeLog	2016-10-24 17:53:12 UTC (rev 207764)
+++ trunk/Source/_javascript_Core/ChangeLog	2016-10-24 17:53:40 UTC (rev 207765)
@@ -1,3 +1,16 @@
+2016-10-24  Mark Lam  <mark....@apple.com>
+
+        Dumping of op_negate bytecode is broken.
+        https://bugs.webkit.org/show_bug.cgi?id=163896
+
+        Reviewed by Darin Adler.
+
+        It thinks the bytecode consists of only 3 machine words, when it consists of 4.
+        This is now fixed.
+
+        * bytecode/CodeBlock.cpp:
+        (JSC::CodeBlock::dumpBytecode):
+
 2016-10-24  Youenn Fablet  <you...@apple.com>
 
         Activate WEB_RTC compilation flags for Mac bots

Modified: trunk/Source/_javascript_Core/bytecode/CodeBlock.cpp (207764 => 207765)


--- trunk/Source/_javascript_Core/bytecode/CodeBlock.cpp	2016-10-24 17:53:12 UTC (rev 207764)
+++ trunk/Source/_javascript_Core/bytecode/CodeBlock.cpp	2016-10-24 17:53:40 UTC (rev 207765)
@@ -999,6 +999,7 @@
         }
         case op_negate: {
             printUnaryOp(out, exec, location, it, "negate");
+            ++it; // op_negate has an extra operand for the ArithProfile.
             break;
         }
         case op_add: {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to