Title: [102485] trunk/Source/_javascript_Core
- Revision
- 102485
- Author
- [email protected]
- Date
- 2011-12-09 15:47:42 -0800 (Fri, 09 Dec 2011)
Log Message
PutByValAlias unnecessarily clobbers GetIndexedPropertyStorage
https://bugs.webkit.org/show_bug.cgi?id=74223
Reviewed by Geoffrey Garen.
Don't clobber GetIndexedPropertyStorage when we see PutByValAlias
* dfg/DFGPropagator.cpp:
(JSC::DFG::Propagator::getIndexedPropertyStorageLoadElimination):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (102484 => 102485)
--- trunk/Source/_javascript_Core/ChangeLog 2011-12-09 23:45:49 UTC (rev 102484)
+++ trunk/Source/_javascript_Core/ChangeLog 2011-12-09 23:47:42 UTC (rev 102485)
@@ -1,3 +1,15 @@
+2011-12-09 Oliver Hunt <[email protected]>
+
+ PutByValAlias unnecessarily clobbers GetIndexedPropertyStorage
+ https://bugs.webkit.org/show_bug.cgi?id=74223
+
+ Reviewed by Geoffrey Garen.
+
+ Don't clobber GetIndexedPropertyStorage when we see PutByValAlias
+
+ * dfg/DFGPropagator.cpp:
+ (JSC::DFG::Propagator::getIndexedPropertyStorageLoadElimination):
+
2011-12-09 David Levin <[email protected]>
Hash* iterators should allow comparison between const and const versions.
Modified: trunk/Source/_javascript_Core/dfg/DFGPropagator.cpp (102484 => 102485)
--- trunk/Source/_javascript_Core/dfg/DFGPropagator.cpp 2011-12-09 23:45:49 UTC (rev 102484)
+++ trunk/Source/_javascript_Core/dfg/DFGPropagator.cpp 2011-12-09 23:47:42 UTC (rev 102485)
@@ -1251,9 +1251,12 @@
// Changing the structure or putting to the storage cannot
// change the property storage pointer.
break;
-
+
+ case PutByValAlias:
+ // PutByValAlias can't change the indexed storage pointer
+ break;
+
case PutByVal:
- case PutByValAlias:
if (isFixedIndexedStorageObjectPrediction(m_graph[node.child1()].prediction()) && byValIsPure(node))
break;
return NoNode;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
