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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to