Changeset 154139 in webkit for trunk/Source/JavaScriptCore

Timestamp:

Aug 15, 2013, 2:54:01 PM (12 years ago)

Author:

[email protected]

Message:

​http://trac.webkit.org/changeset/154120 accidentally changed DFGCapabilities to read the resolve type from operand 4, not 3; it should be 3.

Rubber stamped by Oliver Hunt.

This was causing some test crashes for me.

• dfg/DFGCapabilities.cpp:

(JSC::DFG::capabilityLevel):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• dfg/DFGCapabilities.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-08-15  Filip Pizlo  <[email protected]>
+
+        http://trac.webkit.org/changeset/154120 accidentally changed DFGCapabilities to read the resolve type from operand 4, not 3; it should be 3.
+
+        Rubber stamped by Oliver Hunt.
+        
+        This was causing some test crashes for me.
+
+        * dfg/DFGCapabilities.cpp:
+        (JSC::DFG::capabilityLevel):
+
 2013-08-15  Brent Fulgham  <[email protected]>
 

trunk/Source/JavaScriptCore/dfg/DFGCapabilities.cpp

@@ -184 +184 @@
     case op_resolve_scope: {
         // We don't compile 'catch' or 'with', so there's no point in compiling variable resolution within them.
-        ResolveType resolveType = ResolveModeAndType(pc[4].u.operand).type();
+        ResolveType resolveType = ResolveModeAndType(pc[3].u.operand).type();
         if (resolveType == Dynamic)
             return CannotCompile;