[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