Reviewers: Mads Ager,

Message:
Mads,

may you have a look?

It's almost revert of 4075.  One thing I changed:

diff --git a/src/builtins.cc b/src/builtins.cc
index f5132c6..1e95bc2 100644
--- a/src/builtins.cc
+++ b/src/builtins.cc
@@ -349,7 +349,7 @@ static Object* CallJsBuiltin(const char* name,
   Vector<Object**> argv(Vector<Object**>::New(args.length() - 1));
   int n_args = args.length() - 1;
   for (int i = 0; i < n_args; i++) {
-    argv[i] = &args[i + 1];
+    argv[i] = args.at<Object>(i + 1).location();
   }
   bool pending_exception = false;
   Handle<Object> result = Execution::Call(function,


Description:
Reland 4068 which wrapped Array methods into native builtins (reverted in 4075).

Please review this at http://codereview.chromium.org/857002

Affected files:
  M src/arm/stub-cache-arm.cc
  M src/bootstrapper.cc
  M src/builtins.cc
  M src/contexts.h
  M src/ia32/stub-cache-ia32.cc
  M src/ic.cc
  M src/stub-cache.h
  M src/stub-cache.cc
  M src/top.h
  M src/top.cc
  M src/x64/stub-cache-x64.cc
  M test/cctest/test-serialize.cc


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

Reply via email to