[PATCH] D71982: [docs] Update path to clang-tools-extra
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 31 06:18:53 PST 2019
aaron.ballman added inline comments.
================
Comment at: clang-tools-extra/docs/clang-tidy/Contributing.rst:66
-Once you are done, change to the ``llvm/tools/clang/tools/extra`` directory, and
+Once you are done, change to the ``llvm/clang-tools-extra`` directory, and
let's start!
----------------
Jim wrote:
> I am no sure that "llvm/clang-tools-extra" should be replaced as "llvm-project/clang-tools-extra".
> Maybe someone would confuse with "llvm", "llvm-project" and "llvm-project/llvm"
Elsewhere we use `path/to/llvm/source`, which seems to be sufficiently clear.
================
Comment at: clang-tools-extra/docs/clang-tidy/Contributing.rst:78
:program:`clang-tidy` source code resides in the
-``llvm/tools/clang/tools/extra`` directory and is structured as follows:
+``llvm/clang-tools-extra`` directory and is structured as follows:
----------------
`path/to/llvm/source` here as well.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71982/new/
https://reviews.llvm.org/D71982
More information about the cfe-commits
mailing list