Skip to content

Commit 78e522b

Browse files
kazutakahiratamahesh-attarde
authored andcommitted
[llvm] Proofread BuildingADistribution.rst (llvm#160980)
1 parent b19ab1b commit 78e522b

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

llvm/docs/BuildingADistribution.rst

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ combination of its sub-project tools for distribution. This document covers
1313
useful features of the LLVM build system as well as best practices and general
1414
information 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
1818
workings of the builds described in the :doc:`AdvancedBuilds` document.
1919

2020
General Distribution Guidance
@@ -27,7 +27,7 @@ compiler. This is done so that the compiler you distribute benefits from all the
2727
bug fixes, performance optimizations and general improvements provided by the
2828
new 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
3131
will 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

4242
The guidance for minimizing distribution size is to dynamically link LLVM and
4343
Clang 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
4646
at 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
7676
recommended for building distributions, only installs specific parts of LLVM as
7777
specified 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
8080
tools as the public tools. This can be changed well by setting
8181
*LLVM_INSTALL_TOOLCHAIN_ONLY* to ``On``. The LLVM tools are intended for
8282
development and testing of LLVM, and should only be included in distributions

0 commit comments

Comments
 (0)