Revision: 2689
Author: [email protected]
Date: Fri Aug 14 04:35:10 2009
Log: Refactor SetFunctionInfo to reduce long argument list take 2.

It now includes the change to ARM and x64 architectures as well.

[email protected]
Review URL: http://codereview.chromium.org/165528
http://code.google.com/p/v8/source/detail?r=2689

Modified:
  /branches/bleeding_edge/src/arm/codegen-arm.h
  /branches/bleeding_edge/src/x64/codegen-x64.h

=======================================
--- /branches/bleeding_edge/src/arm/codegen-arm.h       Fri Aug  7 01:18:31 2009
+++ /branches/bleeding_edge/src/arm/codegen-arm.h       Fri Aug 14 04:35:10 2009
@@ -152,14 +152,9 @@
  #endif

    static void SetFunctionInfo(Handle<JSFunction> fun,
-                              int length,
-                              int function_token_position,
-                              int start_position,
-                              int end_position,
-                              bool is_expression,
+                              FunctionLiteral* lit,
                                bool is_toplevel,
-                              Handle<Script> script,
-                              Handle<String> inferred_name);
+                              Handle<Script> script);

    // Accessors
    MacroAssembler* masm() { return masm_; }
=======================================
--- /branches/bleeding_edge/src/x64/codegen-x64.h       Thu Aug  6 04:51:23 2009
+++ /branches/bleeding_edge/src/x64/codegen-x64.h       Fri Aug 14 04:35:10 2009
@@ -299,14 +299,9 @@
  #endif

    static void SetFunctionInfo(Handle<JSFunction> fun,
-                              int length,
-                              int function_token_position,
-                              int start_position,
-                              int end_position,
-                              bool is_expression,
+                              FunctionLiteral* lit,
                                bool is_toplevel,
-                              Handle<Script> script,
-                              Handle<String> inferred_name);
+                              Handle<Script> script);

    // Accessors
    MacroAssembler* masm() { return masm_; }

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to