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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri 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
----------------
I don't think there actually is a "release-with-asserts" build type,
but "relwithdebinfo"?


================
Comment at: llvm/docs/GettingStarted.rst:1126
+   similar to the -j option used with make, but only for link jobs. This option 
+   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 
----------------
Why "of course"?


================
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.
----------------
Since lld is internally parallel, i'd even say this should always be 1 if using lld.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75425