@@ -13,8 +13,8 @@ combination of its sub-project tools for distribution. This document covers
1313useful features of the LLVM build system as well as best practices and general
1414information about packaging LLVM.
1515
16- If you are new to CMake you may find the :doc: `CMake ` or :doc: `CMakePrimer `
17- documentation useful. Some of the things covered in this document are the inner
16+ If you are new to CMake, you may find the :doc: `CMake ` or :doc: `CMakePrimer `
17+ documentation useful. This document covers some of the inner
1818workings of the builds described in the :doc: `AdvancedBuilds ` document.
1919
2020General Distribution Guidance
@@ -27,7 +27,7 @@ compiler. This is done so that the compiler you distribute benefits from all the
2727bug fixes, performance optimizations and general improvements provided by the
2828new compiler.
2929
30- In deciding how to build your distribution there are a few trade-offs that you
30+ In deciding how to build your distribution, there are a few trade-offs that you
3131will need to evaluate. The big two are:
3232
3333#. Compile time of the distribution against performance of the built compiler
@@ -41,8 +41,8 @@ opportunity for the compiler to optimize.
4141
4242The guidance for minimizing distribution size is to dynamically link LLVM and
4343Clang libraries into the tools to reduce code duplication. This will come at a
44- substantial performance penalty to the generated binary both because it reduces
45- optimization opportunity, and because dynamic linking requires resolving symbols
44+ substantial performance penalty to the generated binary, both because it reduces
45+ optimization opportunities and because dynamic linking requires resolving symbols
4646at process launch time, which can be very slow for C++ code.
4747
4848.. _shared_libs :
@@ -76,7 +76,7 @@ LLVM testing tools. Alternatively the ``install-distribution`` target, which is
7676recommended for building distributions, only installs specific parts of LLVM as
7777specified at configuration time by *LLVM_DISTRIBUTION_COMPONENTS *.
7878
79- Additionally by default the ``install `` target will install the LLVM testing
79+ Additionally, by default, the ``install `` target will install the LLVM testing
8080tools as the public tools. This can be changed well by setting
8181*LLVM_INSTALL_TOOLCHAIN_ONLY * to ``On ``. The LLVM tools are intended for
8282development and testing of LLVM, and should only be included in distributions
0 commit comments