[PATCH] D153727: fix: GettingStarted.rst cmake should reference build

Farid Zakaria via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 25 13:48:02 PDT 2023


fzakaria created this revision.
Herald added a project: All.
fzakaria requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The next sections in GettingStarted assume you are still in the root
directory llvm-project when using ninja.

Make the `cmake --build` command match it as well.

Note: I am a new cmake user and this confused me.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D153727

Files:
  llvm/docs/GettingStarted.rst


Index: llvm/docs/GettingStarted.rst
===================================================================
--- llvm/docs/GettingStarted.rst
+++ llvm/docs/GettingStarted.rst
@@ -89,7 +89,7 @@
        you run into memory issues building LLVM, try setting this to limit the
        maximum number of compile/link jobs running at the same time.
 
-   * ``cmake --build . [--target <target>]`` or the build system specified
+   * ``cmake --build build [--target <target>]`` or the build system specified
      above directly.
 
      * The default target (i.e. ``cmake --build .`` or ``make``) will build all of


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153727.534371.patch
Type: text/x-patch
Size: 604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230625/9638878b/attachment.bin>


More information about the llvm-commits mailing list