If you take a look at ConvertArgumentsForLaunchingInShell, I had to make a decision about whether to use -c or /C as an argument to the shell.  I could have string checked the shell name against "cmd.exe", but that seemed more hackish than asking "Is this going to run in a shell on windows"?<br><br><div class="gmail_quote">On Fri Dec 05 2014 at 2:47:11 PM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The only thing I don't know about is the:<br>
<br>
```<br>
    launch_info.SetArchitecture(<u></u>HostInfo::GetArchitecture());<br>
<br>
```<br>
<br>
Line... You might specify a tool on MacOSX that is i386 like "/path/to/i386/a.out" and if you set this to the system architecture x86_64-apple-macosx, it won't execute the shell tool. This should probably be removed. Did you need to do this for windows or did you just add be because you thought it would always be correct?<br>
<br>
<a href="http://reviews.llvm.org/D6553" target="_blank">http://reviews.llvm.org/D6553</a><br>
<br>
<br>
</blockquote></div>