sgraenitz added a comment.

Thanks for your feedback.

In D65330#1604355 <https://reviews.llvm.org/D65330#1604355>, @labath wrote:

> lld is required for running the (dotest) test suite on windows. On other 
> platforms it is not required, but there are tests which will only get run if 
> lld is enabled.


I mentioned this in `Regular in-tree builds` as: //Parts of the LLDB test suite 
require `lld`. Add it to the list in order to run all tests.//



================
Comment at: lldb/docs/resources/build.rst:90
+
+  > git clone https://github.com/llvm/llvm-project.git
 
----------------
jryans wrote:
> This could be done in a separate patch, but I happened to notice that [Get 
> Involved section on the LLDB main 
> page](https://lldb.llvm.org/index.html#get-involved) currently suggests 
> Subversion at the moment. It would be nice to have this updated to Git as 
> well.
+1 (in a separate patch)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65330



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

Reply via email to