Reviewers: fschneider,

Description:
The memory size test makes no sense on the simulated platforms.

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

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     test/cctest/test-mark-compact.cc


Index: test/cctest/test-mark-compact.cc
===================================================================
--- test/cctest/test-mark-compact.cc    (revision 10171)
+++ test/cctest/test-mark-compact.cc    (working copy)
@@ -454,7 +454,10 @@
 }


-#ifdef __linux__
+// Here is a memory use test that uses /proc, and is therefore Linux-only. We +// do not care how much memory the simulator uses, since it is only there for
+// debugging purposes.
+#if defined(__linux__) && !defined(USE_SIMULATOR)


 static uintptr_t ReadLong(char* buffer, intptr_t* position, int base) {
@@ -532,4 +535,4 @@
   }
 }

-#endif  // __linux__
+#endif  // __linux__ and !USE_SIMULATOR


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

Reply via email to