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

[Alexander Kornienko via Phabricator via cfe-commits]

alexfh 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.')
----------------
zinovy.nis wrote:
> 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?
Nope, our CompilationDatabase implementation doesn't support concurrent use (regardless of whether it's the same process or different ones).


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

https://reviews.llvm.org/D57662