Author: [email protected]
Date: Mon May 25 05:35:32 2009
New Revision: 2042
Modified:
branches/bleeding_edge/test/cctest/cctest.status
branches/bleeding_edge/test/cctest/test-log.cc
Log:
Remove ifdef from ProfLazyMode test, instead mark it in cctest.status.
Review URL: http://codereview.chromium.org/113820
Modified: branches/bleeding_edge/test/cctest/cctest.status
==============================================================================
--- branches/bleeding_edge/test/cctest/cctest.status (original)
+++ branches/bleeding_edge/test/cctest/cctest.status Mon May 25 05:35:32
2009
@@ -46,6 +46,9 @@
test-api/OutOfMemory: SKIP
test-api/OutOfMemoryNested: SKIP
+# BUG(355): Test crashes on ARM.
+test-log/ProfLazyMode: SKIP
+
[ $simulator == arm ]
# BUG(271): During exception propagation, we compare pointers into the
Modified: branches/bleeding_edge/test/cctest/test-log.cc
==============================================================================
--- branches/bleeding_edge/test/cctest/test-log.cc (original)
+++ branches/bleeding_edge/test/cctest/test-log.cc Mon May 25 05:35:32 2009
@@ -129,11 +129,6 @@
}
-// Exclude the following test from ARM because profiling can
-// malfunction there.
-// TODO(mikhail.naganov): Fix it.
-#ifndef V8_TARGET_ARCH_ARM
-
namespace v8 {
namespace internal {
@@ -242,8 +237,6 @@
i::FLAG_prof = saved_prof;
i::FLAG_prof_auto = saved_prof_auto;
}
-
-#endif // V8_TARGET_ARCH_ARM
static inline bool IsStringEqualTo(const char* r, const char* s) {
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
