Title: [199070] trunk/Source/_javascript_Core
Revision
199070
Author
[email protected]
Date
2016-04-05 13:05:02 -0700 (Tue, 05 Apr 2016)

Log Message

jsc-layout-tests.yaml/js/script-tests/regress-141098.js failing on Yosemite Debug after r198989
https://bugs.webkit.org/show_bug.cgi?id=156187

Reviewed by Filip Pizlo.

This is a speculative fix. Lets see if the prevents the timeout.

* parser/Parser.cpp:
(JSC::Parser<LexerType>::parseStatementListItem):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (199069 => 199070)


--- trunk/Source/_javascript_Core/ChangeLog	2016-04-05 19:58:04 UTC (rev 199069)
+++ trunk/Source/_javascript_Core/ChangeLog	2016-04-05 20:05:02 UTC (rev 199070)
@@ -1,3 +1,15 @@
+2016-04-05  Saam barati  <[email protected]>
+
+        jsc-layout-tests.yaml/js/script-tests/regress-141098.js failing on Yosemite Debug after r198989
+        https://bugs.webkit.org/show_bug.cgi?id=156187
+
+        Reviewed by Filip Pizlo.
+
+        This is a speculative fix. Lets see if the prevents the timeout.
+
+        * parser/Parser.cpp:
+        (JSC::Parser<LexerType>::parseStatementListItem):
+
 2016-04-04  Filip Pizlo  <[email protected]>
 
         PolymorphicAccess should have a MegamorphicLoad case

Modified: trunk/Source/_javascript_Core/parser/Parser.cpp (199069 => 199070)


--- trunk/Source/_javascript_Core/parser/Parser.cpp	2016-04-05 19:58:04 UTC (rev 199069)
+++ trunk/Source/_javascript_Core/parser/Parser.cpp	2016-04-05 20:05:02 UTC (rev 199070)
@@ -542,6 +542,9 @@
 {
     // The grammar is documented here:
     // http://www.ecma-international.org/ecma-262/6.0/index.html#sec-statements
+
+    failIfStackOverflow();
+
     DepthManager statementDepth(&m_statementDepth);
     m_statementDepth++;
     TreeStatement result = 0;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to