Revision: 18975
Author:   [email protected]
Date:     Fri Jan 31 09:10:56 2014 UTC
Log:      Fix deopt fuzzer variable set up.

Adds the simulator variable configuration from r18959 to the deopt fuzzer.

BUG=
[email protected]

Review URL: https://codereview.chromium.org/143403004
http://code.google.com/p/v8/source/detail?r=18975

Modified:
 /branches/bleeding_edge/tools/run-deopt-fuzzer.py
 /branches/bleeding_edge/tools/run-tests.py
 /branches/bleeding_edge/tools/testrunner/local/utils.py

=======================================
--- /branches/bleeding_edge/tools/run-deopt-fuzzer.py Wed Jan 29 15:30:41 2014 UTC +++ /branches/bleeding_edge/tools/run-deopt-fuzzer.py Fri Jan 31 09:10:56 2014 UTC
@@ -373,6 +373,7 @@
     "isolates": options.isolates,
     "mode": mode,
     "no_i18n": False,
+    "simulator": utils.UseSimulator(arch),
     "system": utils.GuessOS(),
   }
   all_tests = []
=======================================
--- /branches/bleeding_edge/tools/run-tests.py  Thu Jan 30 18:04:33 2014 UTC
+++ /branches/bleeding_edge/tools/run-tests.py  Fri Jan 31 09:10:56 2014 UTC
@@ -388,13 +388,6 @@
                         options.no_i18n)

   # Find available test suites and read test cases from them.
-  machine = platform.machine()
-  if (machine and
-      (arch == "mipsel" or arch == "arm") and
-      not arch.startswith(machine)):
-    use_simulator = True
-  else:
-    use_simulator = False
   variables = {
     "arch": arch,
     "asan": options.asan,
@@ -403,8 +396,8 @@
     "isolates": options.isolates,
     "mode": mode,
     "no_i18n": options.no_i18n,
+    "simulator": utils.UseSimulator(arch),
     "system": utils.GuessOS(),
-    "simulator": use_simulator,
   }
   all_tests = []
   num_tests = 0
=======================================
--- /branches/bleeding_edge/tools/testrunner/local/utils.py Wed Sep 25 12:37:00 2013 UTC +++ /branches/bleeding_edge/tools/testrunner/local/utils.py Fri Jan 31 09:10:56 2014 UTC
@@ -78,6 +78,13 @@
     return None


+def UseSimulator(arch):
+  machine = platform.machine()
+  return (machine and
+      (arch == "mipsel" or arch == "arm") and
+      not arch.startswith(machine))
+
+
 # This will default to building the 32 bit VM even on machines that are
 # capable of running the 64 bit VM.
 def DefaultArch():

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to