[Lldb-commits] [PATCH] D65330: [lldb][docs] Update documentation for monorepo and CMake caches

[Stefan Gränitz via Phabricator via lldb-commits]

sgraenitz added a comment.

In D65330#1602786 <https://reviews.llvm.org/D65330#1602786>, @jryans wrote:

> On http://releases.llvm.org/, for some recent versions like 8.0.0, the "Documentation" column has separate versioned links for "llvm clang lld clang-extra libc++ polly", but lldb doesn't appear there.


Interesting. Thks!



================
Comment at: lldb/docs/resources/build.rst:94
+project, it generates the files needed by your build tool. The recommended
+build tool for LLVM is Ninja. Please also read `Building LLVM with CMake
+<http://llvm.org/docs/CMake.html>`_.
----------------
stella.stamenova wrote:
> Is Ninja the recommended build tool on Windows as well?
Well it's recommended for LLVM and I used it for LLDB on Windows in the past.
Maybe @zturner knows more about it?


================
Comment at: lldb/docs/resources/build.rst:105
+
+  > cmake -G Ninja -DLLVM_ENABLE_PROJECTS="clang;lldb" [<cmake options>] path/to/llvm-project/llvm
+
----------------
stella.stamenova wrote:
> I thought some of the tests need lld as well
So far, the documentation didn't mention lld. I would recommend using lld to **build** LLDB on Linux, but AFAIK it's not required to build lld along with LLDB.


================
Comment at: lldb/docs/resources/build.rst:235
+
+  > cmake -G "Visual Studio 14 2015" <cmake variables> <path to root of llvm source tree>
+
----------------
stella.stamenova wrote:
> We retired VS2015 support
What generator string would you like to see here instead? (So far I just kept what was there.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65330