stella.stamenova added a comment.

In D68980#1709968 <https://reviews.llvm.org/D68980#1709968>, @mstorsjo wrote:

> In D68980#1709954 <https://reviews.llvm.org/D68980#1709954>, 
> @stella.stamenova wrote:
>
> > Both. Again, it's been a long time, but when we use VS for building, the 
> > environment already contains a path to clang-cl so regardless of whether 
> > the build is standalone or not, the build gets confused about which 
> > clang-cl to use. @mstorsjo might want to update the tests to use clang-cl 
> > but then make sure that the update works on the Windows Buildbot (or 
> > similar environment) before committing.
>
>
> Is there any way of checking whether it works on this buildbot, other than 
> committing (after doing best effort testing of it in whatever comparable 
> windows environments I have available)?


Not really. If you have a patch, I can run it for you on my windows environment 
(which should be almost identical to the Buildbot).


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68980/new/

https://reviews.llvm.org/D68980



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to