Commit e8ec4ede authored by chunyang.dai's avatar chunyang.dai Committed by Commit bot

X87: [runtime] Initial step towards switching Execution::Call to callable.

port d5bbd45f (r30808).

oringial commit message:

    Currently Execution::Call (and friends) still duplicate a lot of the
    Call sequence logic that should be encapsulated in the Call and
    CallFunction builtins. So the plan now is to switch Execution::Call
    to accept any Callable and just pass that through to the Call builtin.

BUG=

Review URL: https://codereview.chromium.org/1350183005

Cr-Commit-Position: refs/heads/master@{#30828}
parent 953024c6
...@@ -584,9 +584,7 @@ static void Generate_JSEntryTrampolineHelper(MacroAssembler* masm, ...@@ -584,9 +584,7 @@ static void Generate_JSEntryTrampolineHelper(MacroAssembler* masm,
CallConstructStub stub(masm->isolate(), NO_CALL_CONSTRUCTOR_FLAGS); CallConstructStub stub(masm->isolate(), NO_CALL_CONSTRUCTOR_FLAGS);
__ CallStub(&stub); __ CallStub(&stub);
} else { } else {
ParameterCount actual(eax); __ Call(masm->isolate()->builtins()->Call(), RelocInfo::CODE_TARGET);
__ InvokeFunction(edi, actual, CALL_FUNCTION,
NullCallWrapper());
} }
// Exit the internal frame. Notice that this also removes the empty. // Exit the internal frame. Notice that this also removes the empty.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment