Bug 127201 – CStack Branch: Fix Baseline JIT for X86-32

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Michael Saboff]

This is to get the baseline JIT working again.

[Michael Saboff]

Created attachment 221508 [details]
Patch

[Geoffrey Garen]

Comment on attachment 221508 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=221508&action=review

r=me

> Source/JavaScriptCore/ChangeLog:9
> +        epilogues) and making room on the stack for outgoiing arguments.

Should be "outgoing".

> Source/JavaScriptCore/jit/ThunkGenerators.cpp:523
> +    // Install the new return PC.
> +    // FIXME: I don't think currentReturnThunkPC is used and should be deleted.
> +#  if 0
> +    jit.loadPtr(&vm->currentReturnThunkPC, GPRInfo::regT2);
> +    jit.storePtr(GPRInfo::regT2, JSInterfaceJIT::Address(JSInterfaceJIT::callFrameRegister, CallFrame::returnPCOffset()));
> +#   else
> +    jit.storePtr(GPRInfo::regT5, JSInterfaceJIT::Address(JSInterfaceJIT::callFrameRegister, CallFrame::returnPCOffset()));
> +#   endif

Need a bug about this.

[Michael Saboff]

(In reply to comment #2)
> (From update of attachment 221508 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=221508&action=review
> 
> r=me
> 
> > Source/JavaScriptCore/ChangeLog:9
> > +        epilogues) and making room on the stack for outgoiing arguments.
> 
> Should be "outgoing".

Fixed.
 
> > Source/JavaScriptCore/jit/ThunkGenerators.cpp:523
> > +    // Install the new return PC.
> > +    // FIXME: I don't think currentReturnThunkPC is used and should be deleted.
> > +#  if 0
> > +    jit.loadPtr(&vm->currentReturnThunkPC, GPRInfo::regT2);
> > +    jit.storePtr(GPRInfo::regT2, JSInterfaceJIT::Address(JSInterfaceJIT::callFrameRegister, CallFrame::returnPCOffset()));
> > +#   else
> > +    jit.storePtr(GPRInfo::regT5, JSInterfaceJIT::Address(JSInterfaceJIT::callFrameRegister, CallFrame::returnPCOffset()));
> > +#   endif
> 
> Need a bug about this.

Added <https://bugs.webkit.org/show_bug.cgi?id=127205> - CStack Branch: VM::currentReturnThunkPC appears to be unused and should be removed

[Michael Saboff]

Committed r162242: <http://trac.webkit.org/changeset/162242>