Author: [EMAIL PROTECTED]
Date: Thu Sep 11 21:31:34 2008
New Revision: 290
Modified:
branches/bleeding_edge/src/builtins-arm.cc
branches/bleeding_edge/src/top.cc
branches/bleeding_edge/test/mjsunit/mjsunit.status
Log:
- 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
Modified: branches/bleeding_edge/src/builtins-arm.cc
==============================================================================
--- branches/bleeding_edge/src/builtins-arm.cc (original)
+++ branches/bleeding_edge/src/builtins-arm.cc Thu Sep 11 21:31:34 2008
@@ -644,7 +644,7 @@
{ 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);
Modified: branches/bleeding_edge/src/top.cc
==============================================================================
--- branches/bleeding_edge/src/top.cc (original)
+++ branches/bleeding_edge/src/top.cc Thu Sep 11 21:31:34 2008
@@ -621,9 +621,6 @@
}
-// 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
Modified: branches/bleeding_edge/test/mjsunit/mjsunit.status
==============================================================================
--- branches/bleeding_edge/test/mjsunit/mjsunit.status (original)
+++ branches/bleeding_edge/test/mjsunit/mjsunit.status Thu Sep 11 21:31:34
2008
@@ -62,6 +62,6 @@
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
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
