[PATCH] D57662: [clang-tidy] Parallelise clang-tidy-diff.py

Zinovy Nis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 15 06:40:25 PST 2019


zinovy.nis marked an inline comment as done.
zinovy.nis added inline comments.


================
Comment at: clang-tidy/tool/clang-tidy-diff.py:98
 
+  parser.add_argument('-j', type=int, default=0,
+                      help='number of tidy instances to be run in parallel.')
----------------
alexfh wrote:
> The "clang-tidy runs are independent" assumption is unfortunately not valid for our internal compilation database integration, so making `-j0` the default will break it. Let's make it `1`.
Do you mean you run it with `-fix` or like this?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57662/new/

https://reviews.llvm.org/D57662





More information about the cfe-commits mailing list