Title: [212957] branches/safari-603.1.30.0-branch/Source/_javascript_Core
Revision
212957
Author
matthew_han...@apple.com
Date
2017-02-24 09:02:09 -0800 (Fri, 24 Feb 2017)

Log Message

rdar://problem/30675867

Modified Paths


Diff

Modified: branches/safari-603.1.30.0-branch/Source/_javascript_Core/ChangeLog (212956 => 212957)


--- branches/safari-603.1.30.0-branch/Source/_javascript_Core/ChangeLog	2017-02-24 16:59:39 UTC (rev 212956)
+++ branches/safari-603.1.30.0-branch/Source/_javascript_Core/ChangeLog	2017-02-24 17:02:09 UTC (rev 212957)
@@ -1,5 +1,15 @@
 2017-02-23  Filip Pizlo  <fpi...@apple.com>
 
+        verifyEdges should not run in release
+        <rdar://problem/30537798>
+
+        Reviewed by Keith Miller and Mark Lam.
+
+        * dfg/DFGAbstractInterpreterInlines.h:
+        (JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):
+
+2017-02-23  Filip Pizlo  <fpi...@apple.com>
+
         Disable concurrent GC A:B testing.
 
         * runtime/Options.h:

Modified: branches/safari-603.1.30.0-branch/Source/_javascript_Core/dfg/DFGAbstractInterpreterInlines.h (212956 => 212957)


--- branches/safari-603.1.30.0-branch/Source/_javascript_Core/dfg/DFGAbstractInterpreterInlines.h	2017-02-24 16:59:39 UTC (rev 212956)
+++ branches/safari-603.1.30.0-branch/Source/_javascript_Core/dfg/DFGAbstractInterpreterInlines.h	2017-02-24 17:02:09 UTC (rev 212957)
@@ -172,7 +172,8 @@
 template<typename AbstractStateType>
 bool AbstractInterpreter<AbstractStateType>::executeEffects(unsigned clobberLimit, Node* node)
 {
-    verifyEdges(node);
+    if (!ASSERT_DISABLED)
+        verifyEdges(node);
     
     m_state.createValueForNode(node);
     
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to