[PATCH] D75425: [docs] Added solutions to slow build under common problems

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

@e-leclercq Would you create a new diff to address the new comments?



================
Comment at: llvm/docs/GettingStarted.rst:1110
+   performance. In the case that you are compiling lld, you may wish to use the
+   gold linker as a faster alternative.
+
----------------
thakis wrote:
> This is misleading. You should always use lld, even when working on lld. Lld is faster at linking lld than gold is. This should recommend lld over gold more strongly, and possibly not mention gold at all -- I'm aware of no reason to use gold (except possibly if you're on a mips box).
One reason to not use lld is when building LLVM the first time on a system that does not have lld installed (or available as binary in a package repository).


================
Comment at: llvm/docs/GettingStarted.rst:1134
+ * -DLLVM_OPTIMIZED_TABLEGEN
+   Set this to ON to generate a fully optimized tablegen during build. This will
+   significantly improve your build time.
----------------
thakis wrote:
> I believe this is only useful if you're doing debug builds, which you already advise against.
You need a debug build for debugging, when printf-debugging is not enough. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75425