Title: [154108] trunk/Source/_javascript_Core
Revision
154108
Author
[email protected]
Date
2013-08-15 10:23:39 -0700 (Thu, 15 Aug 2013)

Log Message

Remove bogus assertion.

Modified Paths


Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (154107 => 154108)


--- trunk/Source/_javascript_Core/ChangeLog	2013-08-15 16:53:20 UTC (rev 154107)
+++ trunk/Source/_javascript_Core/ChangeLog	2013-08-15 17:23:39 UTC (rev 154108)
@@ -1,3 +1,12 @@
+2013-08-15  Oliver Hunt  <[email protected]>
+
+        Remove bogus assertion.
+
+        RS=Filip Pizlo
+
+        * dfg/DFGAbstractInterpreterInlines.h:
+        (JSC::DFG::::executeEffects):
+
 2013-08-15  Allan Sandfeld Jensen  <[email protected]>
 
         REGRESSION(r148790) Made 7 tests fail on x86 32bit

Modified: trunk/Source/_javascript_Core/dfg/DFGAbstractInterpreterInlines.h (154107 => 154108)


--- trunk/Source/_javascript_Core/dfg/DFGAbstractInterpreterInlines.h	2013-08-15 16:53:20 UTC (rev 154107)
+++ trunk/Source/_javascript_Core/dfg/DFGAbstractInterpreterInlines.h	2013-08-15 17:23:39 UTC (rev 154108)
@@ -1274,16 +1274,6 @@
     case StructureTransitionWatchpoint: {
         AbstractValue& value = forNode(node->child1());
 
-        // It's only valid to issue a structure transition watchpoint if we already
-        // know that the watchpoint covers a superset of the structures known to
-        // belong to the set of future structures that this value may have.
-        // Currently, we only issue singleton watchpoints (that check one structure)
-        // and our futurePossibleStructure set can only contain zero, one, or an
-        // infinity of structures.
-        ASSERT(
-            value.m_futurePossibleStructure.isSubsetOf(StructureSet(node->structure()))
-            || m_graph.watchpoints().shouldAssumeMixedState(node->structure()->transitionWatchpointSet()));
-        
         filter(value, node->structure());
         m_state.setHaveStructures(true);
         node->setCanExit(true);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to