HomePhabricator

[docs] Added solutions to slow build under common problems.

Authored by e-leclercq on Mar 28 2020, 2:09 AM.

Description

[docs] Added solutions to slow build under common problems.

I added a list of options to configure should someone have issues with
long build time or running out of memory. This was added under common
problems in the getting started section of the documentation.

Reviewed By: Meinersbur, dim, e-leclercq

Differential Revision: https://reviews.llvm.org/D75425

Details

Committed
MeinersburMar 28 2020, 2:19 AM
Reviewer
Meinersbur
Differential Revision
D75425: [docs] Added solutions to slow build under common problems
Parents
rG65b4695375c6: tsan: fix build bugs
Branches
Unknown
Tags
Unknown

Event Timeline

nikic added inline comments.
/llvm/docs/GettingStarted.rst
1144

Tried this just now and got an error. You are required to also set -DCLANG_ENABLE_ARCMT=OFF.

Unfortunately I haven't found any information /t all about what arcmt is (or what the abbreviation stands for) or why it is part of a default clang build.

Pinging @rjmccall as the primary author of this code: Would it be possible to add some comments at the top of https://github.com/llvm/llvm-project/blob/master/clang/lib/ARCMigrate/ARCMT.cpp or next to the CLANG_ENABLE_ARMCT CMake option about what this functionality does? It's hard to decide whether this is safe to disable right now.

I am not actually the primary author of that code; pinging @akyrtzi. The ARC migrator is a source tool which IIRC is built on top of the static analyzer, which is presumably why it fails to build if you disable the static analyzer. I think it would be reasonable to either disable it by default if you disable the static analyzer or to at least emit a clearer error about the inconsistency.

I agree it makes sense to disable CLANG_ENABLE_ARCMT if CLANG_ENABLE_STATIC_ANALYZER is also disabled.

foad added a subscriber: foad.Mar 31 2020, 2:00 AM
foad added inline comments.
/llvm/docs/GettingStarted.rst
1123

I don't think "of course" is justified here!

danilaml added inline comments.
/llvm/docs/GettingStarted.rst
1110

Is gold really a faster alternative to lld?