Changeset 39156 in webkit for trunk/JavaScriptCore/interpreter/Interpreter.h

Timestamp:

Dec 9, 2008, 4:26:13 PM (16 years ago)

Author:

[email protected]

Message:

2008-12-09 Sam Weinig <​[email protected]>

Reviewed by Geoffrey Garen.

Remove unnecessary extra lookup when throwing an exception.
We used to first lookup the target offset using getHandlerForVPC
and then we would lookup the native code stub using
nativeExceptionCodeForHandlerVPC. Instead, we can just pass around
the HandlerInfo.

• bytecode/CodeBlock.cpp: (JSC::CodeBlock::handlerForVPC): Return the HandlerInfo.
• bytecode/CodeBlock.h: Remove nativeExceptionCodeForHandlerVPC.

• interpreter/Interpreter.cpp: (JSC::Interpreter::throwException): Return a HandlerInfo instead of and Instruction offset. (JSC::Interpreter::privateExecute): Get the offset from HandlerInfo. (JSC::Interpreter::cti_op_throw): Get the native code from the HandleInfo. (JSC::Interpreter::cti_vm_throw): Ditto.
• interpreter/Interpreter.h:

File:

• trunk/JavaScriptCore/interpreter/Interpreter.h (modified) (2 diffs)

trunk/JavaScriptCore/interpreter/Interpreter.h

@@ -51 +51 @@
     class ScopeChainNode;
     class SamplingTool;
+    struct HandlerInfo;
 
 #if ENABLE(JIT)
@@ -302 +303 @@
 
         NEVER_INLINE bool unwindCallFrame(CallFrame*&, JSValue*, const Instruction*&, CodeBlock*&);
-        NEVER_INLINE Instruction* throwException(CallFrame*&, JSValue*&, const Instruction*, bool);
+        NEVER_INLINE HandlerInfo* throwException(CallFrame*&, JSValue*&, const Instruction*, bool);
         NEVER_INLINE bool resolveBaseAndFunc(CallFrame*, Instruction*, JSValue*& exceptionValue);