Reviewers: dstence, joransiu, john.yan, rmcilroy,

Description:
PPC: [Interpreter] Add support for parameter variables.

Port 5d975694e4d3ecf66716cc5395d4d10c9730f9dd

Original commit message:
    Adds support for parameters to the BytecodeArrayBuilder and
BytecodeGenerator.
    Parameters are accessed as negative interpreter registers.

[email protected], [email protected], [email protected],
[email protected]
BUG=v8:4280
LOG=N

Please review this at https://codereview.chromium.org/1308373003/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+5, -2 lines):
  M src/ppc/builtins-ppc.cc


Index: src/ppc/builtins-ppc.cc
diff --git a/src/ppc/builtins-ppc.cc b/src/ppc/builtins-ppc.cc
index bf65eaa69a90d4d97c0754c3c7895eb762ee35b9..96c497697bec06f10582d1677ac966c5d2dc10af 100644
--- a/src/ppc/builtins-ppc.cc
+++ b/src/ppc/builtins-ppc.cc
@@ -991,8 +991,11 @@ void Builtins::Generate_InterpreterExitTrampoline(MacroAssembler* masm) {

   // Leave the frame (also dropping the register file).
   __ LeaveFrame(StackFrame::JAVA_SCRIPT);
-  // Drop receiver + arguments.
- __ Drop(1); // TODO(rmcilroy): Get number of arguments from BytecodeArray.
+
+  // Drop receiver + arguments and return.
+  __ lwz(r0, FieldMemOperand(kInterpreterBytecodeArrayRegister,
+                             BytecodeArray::kParameterSizeOffset));
+  __ add(sp, sp, r0);
   __ blr();
 }



--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to