[test-suite] r256944 - [test-suite] This change causes lit to use the host-compiled timeit when not

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 6 08:04:40 PST 2016


Author: mcrosier
Date: Wed Jan  6 10:04:40 2016
New Revision: 256944

URL: http://llvm.org/viewvc/llvm-project?rev=256944&view=rev
Log:
[test-suite] This change causes lit to use the host-compiled timeit when not
remoting into another host to run the test suite. timeit-target is intended to
be used on a host with the target architecture if cross-compiling, we should
always use timeit on the build host.

This is most important for cross-compiling and running under an
emulator, as timeit wraps around any run-under scripts given to lit.

http://reviews.llvm.org/D15870

Modified:
    test-suite/trunk/lit.cfg

Modified: test-suite/trunk/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/lit.cfg?rev=256944&r1=256943&r2=256944&view=diff
==============================================================================
--- test-suite/trunk/lit.cfg (original)
+++ test-suite/trunk/lit.cfg Wed Jan  6 10:04:40 2016
@@ -116,7 +116,9 @@ def prepareRunSafely(config, commandline
     runsafely_prefix = [ runsafely ]
     if workdir is not None:
         runsafely_prefix += [ "-d", workdir ]
+    timeit = "%s/tools/timeit" % config.test_source_root
     if config.remote_host:
+        timeit = "%s/tools/timeit-target" % config.test_source_root
         runsafely_prefix += [ "-r", config.remote_host ]
         if config.remote_user:
             runsafely_prefix += [ "-l", config.remote_user ]
@@ -135,7 +137,6 @@ def prepareRunSafely(config, commandline
     else:
         if stdout is not None or stderr is not None:
             raise Exception("separate stdout/stderr redirection not possible with traditional output")
-    timeit = "%s/tools/timeit-target" % config.test_source_root
     timeout = "7200"
     if stdin is None:
         stdin = "/dev/null"




More information about the llvm-commits mailing list