Skip to content

Commit

Permalink
[Docs] Minor Fixups in Advanced Builds Documentation
Browse files Browse the repository at this point in the history
This patch changes two instances of an ampersand to a written out and
for more consistency with the rest of the file and brevity. In addition,
the last `cmake --build` reference is removed, again for consistency
with the rest of the file which shows the ninja invocations. This cmake
invocation also passed in the `--parallel` flag which doesn't make sense
with ninja using all threads by default.

This was changed in the previous patch to touch this line
(https://reviews.llvm.org/D88990), but if we want to change this, it
should be done across the entire file.
  • Loading branch information
boomanaiden154 committed May 13, 2023
1 parent e36caae commit 1a83865
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions llvm/docs/AdvancedBuilds.rst
Expand Up @@ -270,12 +270,12 @@ and build a compiler (stage1), then use that compiler to rebuild the sources
this, you have a stage2 and stage3 compiler that should be bit-for-bit
identical.

You can perform one of these 3-stage builds with LLVM & clang using the
You can perform one of these 3-stage builds with LLVM and clang using the
following commands:

.. code-block:: console
$ cmake -G Ninja -C <path to source>/clang/cmake/caches/3-stage.cmake <path to source>/llvm
$ cmake --build . --target stage3 --parallel
$ ninja stage3
After the build you can compare the stage2 & stage3 compilers.
After the build you can compare the stage2 and stage3 compilers.

0 comments on commit 1a83865

Please sign in to comment.