Changeset 156242 in webkit for trunk/Source/JavaScriptCore/jit/JITExceptions.cpp

Timestamp:

Sep 21, 2013, 10:00:45 PM (12 years ago)

Author:

[email protected]

Message:

Interpreter::unwind() has no need for the bytecodeOffset
​https://bugs.webkit.org/show_bug.cgi?id=121755

Reviewed by Oliver Hunt.

It was only using the bytecodeOffset for some debugger stuff, but the debugger could
just get the bytecodeOffset the same way the rest of the machinery does: by using the
CallFrame's location.

It turns out that a lot of really ugly code was in place just to supply this
bytecodeOffset. This patch kills most of that code, and allows us to kill even more
code in a future patch - though most likely that killage will involve further
refactorings as well, see ​https://bugs.webkit.org/show_bug.cgi?id=121734.

• dfg/DFGOperations.cpp:
• interpreter/CallFrame.cpp:

(JSC::CallFrame::bytecodeOffset):
(JSC::CallFrame::codeOrigin):

• interpreter/CallFrame.h:
• interpreter/Interpreter.cpp:

(JSC::Interpreter::unwind):

• interpreter/Interpreter.h:
• jit/JITExceptions.cpp:

(JSC::genericUnwind):

• jit/JITExceptions.h:
• jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):
(JSC::cti_vm_handle_exception):

• llint/LLIntExceptions.cpp:

(JSC::LLInt::doThrow):
(JSC::LLInt::returnToThrow):
(JSC::LLInt::callToThrow):

• llint/LLIntExceptions.h:
• llint/LLIntSlowPaths.cpp:

(JSC::LLInt::LLINT_SLOW_PATH_DECL):

• runtime/CommonSlowPaths.cpp:

(JSC::SLOW_PATH_DECL):

• runtime/CommonSlowPathsExceptions.cpp:

(JSC::CommonSlowPaths::interpreterThrowInCaller):

• runtime/CommonSlowPathsExceptions.h:

File:

• trunk/Source/JavaScriptCore/jit/JITExceptions.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/jit/JITExceptions.cpp

@@ -39 +39 @@
 namespace JSC {
 
-static unsigned getExceptionLocation(VM* vm, CallFrame* callFrame)
-{
-    UNUSED_PARAM(vm);
-    ASSERT(!callFrame->hasHostCallFrameFlag());
-
-#if ENABLE(DFG_JIT)
-    if (callFrame->hasLocationAsCodeOriginIndex())
-        return callFrame->bytecodeOffsetFromCodeOriginIndex();
-#endif
-
-    return callFrame->locationAsBytecodeOffset();
-}
-
 #if USE(JSVALUE32_64)
 EncodedExceptionHandler encode(ExceptionHandler handler)
@@ -68 +55 @@
 }
 
-ExceptionHandler genericUnwind(VM* vm, ExecState* callFrame, JSValue exceptionValue, unsigned vPCIndex)
+ExceptionHandler genericUnwind(VM* vm, ExecState* callFrame, JSValue exceptionValue)
 {
     RELEASE_ASSERT(exceptionValue);
-    HandlerInfo* handler = vm->interpreter->unwind(callFrame, exceptionValue, vPCIndex); // This may update callFrame.
+    HandlerInfo* handler = vm->interpreter->unwind(callFrame, exceptionValue); // This may update callFrame.
 
     void* catchRoutine;
@@ -90 +77 @@
 }
 
-ExceptionHandler jitThrowNew(VM* vm, ExecState* callFrame, JSValue exceptionValue)
-{
-    unsigned bytecodeOffset = getExceptionLocation(vm, callFrame);
-    
-    return genericUnwind(vm, callFrame, exceptionValue, bytecodeOffset);
-}
-
-ExceptionHandler jitThrow(VM* vm, ExecState* callFrame, JSValue exceptionValue, ReturnAddressPtr faultLocation)
-{
-    return genericUnwind(vm, callFrame, exceptionValue, callFrame->codeBlock()->bytecodeOffset(callFrame, faultLocation));
-}
-
 }