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

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added inline comments.


================
Comment at: llvm/docs/GettingStarted.rst:1118
+   the linking phase. So, you may wish to use the build type Release. Another 
+   build type you may wish to consider is release-with-asserts which compiles at 
+   nearly the same rate as the Release build; however, it may not be as easy
----------------
lebedev.ri wrote:
> I don't think there actually is a "release-with-asserts" build type,
> but "relwithdebinfo"?
What is meant here is `-DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=ON`.

I don't think `relwithdebinfo` solves any problem compiling LLVM itself. Files will still be large because they contain debug info.


================
Comment at: llvm/docs/GettingStarted.rst:1127-1128
+   is of course only meaningful if you plan to build with ninja. You may wish to 
+   use a very low number of jobs, as this will greatly reduce the amount memory 
+   used during the build process. If you have limited memory, you may wish to 
+   set this to 1.
----------------
lebedev.ri wrote:
> Since lld is internally parallel, i'd even say this should always be 1 if using lld.
If this is true, our CMakeLists.txt should set `LLVM_PARALLEL_LINK_JOBS` whenever `LLVM_USE_LINKER` is set to lld.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75425