Title: [129325] trunk/Source/_javascript_Core
- Revision
- 129325
- Author
- [email protected]
- Date
- 2012-09-23 20:47:49 -0700 (Sun, 23 Sep 2012)
Log Message
PutScopedVar should not be marked as clobbering the world
https://bugs.webkit.org/show_bug.cgi?id=97416
Reviewed by Filip Pizlo.
No performance change.
PutScopedVar doesn't have arbitrary side-effects, so it shouldn't be marked
as such.
* dfg/DFGNodeType.h:
(DFG):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (129324 => 129325)
--- trunk/Source/_javascript_Core/ChangeLog 2012-09-24 03:35:18 UTC (rev 129324)
+++ trunk/Source/_javascript_Core/ChangeLog 2012-09-24 03:47:49 UTC (rev 129325)
@@ -1,5 +1,20 @@
2012-09-23 Geoffrey Garen <[email protected]>
+ PutScopedVar should not be marked as clobbering the world
+ https://bugs.webkit.org/show_bug.cgi?id=97416
+
+ Reviewed by Filip Pizlo.
+
+ No performance change.
+
+ PutScopedVar doesn't have arbitrary side-effects, so it shouldn't be marked
+ as such.
+
+ * dfg/DFGNodeType.h:
+ (DFG):
+
+2012-09-23 Geoffrey Garen <[email protected]>
+
I accidentally the whole 32-bit :(.
Unbreak the DFG in 32-bit with the 32-bit path I forgot in my last patch.
Modified: trunk/Source/_javascript_Core/dfg/DFGNodeType.h (129324 => 129325)
--- trunk/Source/_javascript_Core/dfg/DFGNodeType.h 2012-09-24 03:35:18 UTC (rev 129324)
+++ trunk/Source/_javascript_Core/dfg/DFGNodeType.h 2012-09-24 03:47:49 UTC (rev 129325)
@@ -147,7 +147,7 @@
macro(GetScope, NodeResultJS) \
macro(GetScopeRegisters, NodeResultStorage) \
macro(GetScopedVar, NodeResultJS) \
- macro(PutScopedVar, NodeMustGenerate | NodeClobbersWorld) \
+ macro(PutScopedVar, NodeMustGenerate) \
macro(GetGlobalVar, NodeResultJS) \
macro(PutGlobalVar, NodeMustGenerate) \
macro(GlobalVarWatchpoint, NodeMustGenerate) \
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
