It’s possible we lost some environment variable propagation, that would do it.  But I’m curious how it was finding the visual studio installation before my patch.<br><br>It also looks like it’s failing finding link.exe (we really should make lld-link the default).  Another fix is to pass -fuse-ld=lld or split the clang-cl line to separate compiler and linker invocations <br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 7, 2018 at 1:44 PM Stella Stamenova 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">stella.stamenova added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D54009#1290644" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54009#1290644</a>, @zturner wrote:<br>
<br>
> I have not run the dotest suite recently, is that where you’re seeing the<br>
>  failures? I successfully ran the lit suite and unit tests though<br>
<br>
<br>
Yes, they are primarily in the lldb-suite but not only:<br>
<br>
<a href="http://lab.llvm.org:8014/builders/lldb-x64-windows-ninja/builds/1129/steps/test/logs/stdio" rel="noreferrer" target="_blank">http://lab.llvm.org:8014/builders/lldb-x64-windows-ninja/builds/1129/steps/test/logs/stdio</a><br>
<br>
There are some other failures, but what I suspect happened after this change is that the tests are now not picking up clang-cl correctly:<br>
<br>
  $ "clang-cl" "/Zi" "E:\build_slave\lldb-x64-windows-ninja\llvm\tools\lldb\lit\Expr/Inputs/call-function.cpp" "-o" "E:\build_slave\lldb-x64-windows-ninja\build\tools\lldb\lit\Expr\Output\TestIRMemoryMapWindows.test.tmp"<br>
  # command stderr:<br>
  clang-cl: warning: unable to find a Visual Studio installation; try running Clang from a developer command prompt [-Wmsvc-not-found]<br>
<br>
  clang-cl: error: unable to execute command: program not executable<br>
<br>
  clang-cl: error: linker command failed with exit code 1 (use -v to see invocation)<br>
<br>
I will have some time to have a look in more detail tomorrow.<br>
<br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
<a href="https://reviews.llvm.org/D54009" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54009</a><br>
<br>
<br>
<br>
</blockquote></div>