Lgtm<br><div class="gmail_quote"><div dir="ltr">On Fri, Feb 3, 2017 at 6:49 PM Marcos Pividori via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mpividori created this revision.<br class="gmail_msg">
<br class="gmail_msg">
This configuration is necessary, and is included in all the tests suites, like the one for clang, for the main llvm repository, etc.<br class="gmail_msg">
We need to execute: `config.test_format = lit.formats.ShTest(False)`<br class="gmail_msg">
Otherwise, lit will try to use bash, which generated the problems mentioned in: <a href="https://reviews.llvm.org/D29515" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D29515</a>  , or cmd prompt which generates a lot of problems.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D29529" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D29529</a><br class="gmail_msg">
<br class="gmail_msg">
Files:<br class="gmail_msg">
  lib/Fuzzer/test/lit.cfg<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
Index: lib/Fuzzer/test/lit.cfg<br class="gmail_msg">
===================================================================<br class="gmail_msg">
--- lib/Fuzzer/test/lit.cfg<br class="gmail_msg">
+++ lib/Fuzzer/test/lit.cfg<br class="gmail_msg">
@@ -6,6 +6,23 @@<br class="gmail_msg">
 config.suffixes = ['.test']<br class="gmail_msg">
 config.test_source_root = os.path.dirname(__file__)<br class="gmail_msg">
<br class="gmail_msg">
+# Choose between lit's internal shell pipeline runner and a real shell.  If<br class="gmail_msg">
+# LIT_USE_INTERNAL_SHELL is in the environment, we use that as an override.<br class="gmail_msg">
+use_lit_shell = os.environ.get("LIT_USE_INTERNAL_SHELL")<br class="gmail_msg">
+if use_lit_shell:<br class="gmail_msg">
+    # 0 is external, "" is default, and everything else is internal.<br class="gmail_msg">
+    execute_external = (use_lit_shell == "0")<br class="gmail_msg">
+else:<br class="gmail_msg">
+    # Otherwise we default to internal on Windows and external elsewhere, as<br class="gmail_msg">
+    # bash on Windows is usually very slow.<br class="gmail_msg">
+    execute_external = (not sys.platform in ['win32'])<br class="gmail_msg">
+<br class="gmail_msg">
+# testFormat: The test format to use to interpret tests.<br class="gmail_msg">
+#<br class="gmail_msg">
+# For now we require '&&' between commands, until they get globally killed and<br class="gmail_msg">
+# the test runner updated.<br class="gmail_msg">
+config.test_format = lit.formats.ShTest(execute_external)<br class="gmail_msg">
+<br class="gmail_msg">
 # Tweak PATH to include llvm tools dir and current exec dir.<br class="gmail_msg">
 llvm_tools_dir = getattr(config, 'llvm_tools_dir', None)<br class="gmail_msg">
 if (not llvm_tools_dir) or (not os.path.exists(llvm_tools_dir)):<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>