Changeset 36604 in webkit for trunk/JavaScriptCore/VM/Machine.h

Timestamp:

Sep 17, 2008, 11:20:42 PM (17 years ago)

Author:

[email protected]

Message:

2008-09-17 Cameron Zwarich <​[email protected]>

Reviewed by Maciej Stachowiak.

Bug 20876: REGRESSION (r36417, r36427): fast/js/exception-expression-offset.html fails
<​https://bugs.webkit.org/show_bug.cgi?id=20876>

r36417 and r36427 caused an get_by_id opcode to be emitted before the
instanceof and construct opcodes, in order to enable inline caching of
the prototype property. Unfortunately, this regressed some tests dealing
with exceptions thrown by 'instanceof' and the 'new' operator. We fix
these problems by detecting whether an "is not an object" exception is
thrown before op_instanceof or op_construct, and emit the proper
exception in those cases.

• VM/CodeGenerator.cpp: (JSC::CodeGenerator::emitConstruct):
• VM/CodeGenerator.h:
• VM/ExceptionHelpers.cpp: (JSC::createInvalidParamError): (JSC::createNotAConstructorError): (JSC::createNotAnObjectError):
• VM/ExceptionHelpers.h:
• VM/Machine.cpp: (JSC::Machine::getOpcode): (JSC::Machine::privateExecute):
• VM/Machine.h:
• kjs/nodes.cpp: (JSC::NewExprNode::emitCode): (JSC::InstanceOfNode::emitCode):

File:

• trunk/JavaScriptCore/VM/Machine.h (modified) (2 diffs)

trunk/JavaScriptCore/VM/Machine.h

@@ -75 +75 @@
         RegisterFile& registerFile() { return m_registerFile; }
         
-        Opcode getOpcode(OpcodeID id)
-        {
-            #if HAVE(COMPUTED_GOTO)
-                return m_opcodeTable[id];
-            #else
-                return id;
-            #endif
-        }
+        static Opcode getOpcode(OpcodeID id);
 
         OpcodeID getOpcodeID(Opcode opcode)
@@ -300 +293 @@
 
 #if HAVE(COMPUTED_GOTO)
-        Opcode m_opcodeTable[numOpcodeIDs]; // Maps OpcodeID => Opcode for compiling
+        static Opcode s_opcodeTable[numOpcodeIDs]; // Maps OpcodeID => Opcode for compiling
         HashMap<Opcode, OpcodeID> m_opcodeIDTable; // Maps Opcode => OpcodeID for decompiling
 #endif