- Update test status file with fixed tests for ARM and add failing test tracked by issue 67.

- Remove debugging aid stop("Generate_ArgumentsAdaptorTrampoline - non-function call")
- Cleanup comment

Review URL: http://codereview.chromium.org/2801

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@290 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

src/builtins-arm.cc

@@ -644,7 +644,7 @@ void Builtins::Generate_ArgumentsAdaptorTrampoline(MacroAssembler* masm) {
   { Label invoke;
     __ tst(r1, r1);
     __ b(ne, &invoke);
-    __ stop("Generate_ArgumentsAdaptorTrampoline - non-function call");
+    //__ stop("Generate_ArgumentsAdaptorTrampoline - non-function call");
     __ mov(r2, Operand(0));  // expected arguments is 0 for CALL_NON_FUNCTION
     __ GetBuiltinEntry(r3, Builtins::CALL_NON_FUNCTION);
     __ b(&enough);

src/top.cc

@@ -621,9 +621,6 @@ Object* Top::PromoteScheduledException() {
 }
 
 
-// TODO(1233523): Get rid of this hackish abstraction once all
-// JavaScript frames have a function associated with them.
-
 // NOTE: The stack trace frame iterator is an iterator that only
 // traverse proper JavaScript frames; that is JavaScript frames that
 // have proper JavaScript functions. This excludes the problematic

test/mjsunit/mjsunit.status

@@ -62,6 +62,6 @@ debug-step-stub-callfunction: FAIL
 debug-stepin-constructor: FAIL
 debug-step: FAIL
 regress/regress-998565: FAIL
-regress/regress-1081309: FAIL
-# Call as function does not always work on ARM port yet.
-number-string-index-call: FAIL
+
+# Issue 67: ARM fails in: Generate_ArgumentsAdaptorTrampoline - non-function call
+call-non-function-call: FAIL