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

Zinovy Nis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 22 00:17:39 PST 2019


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


================
Comment at: clang-tidy/tool/clang-tidy-diff.py:133
+    if args.export_fixes:
+      print("error: -export-fixes and -j are mutually exclusive.")
+      sys.exit(1)
----------------
alexfh wrote:
> What's wrong with -export-fixes and -j? Different clang-tidy processes could export the fixes to different files (e.g. the -export-fixes= value could be used as a prefix of the filename with a sequential or random suffix. WDYT?
I can copy/paste (shame on me!) a code from `run-clang-tidy.py` that does just what we need.


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

https://reviews.llvm.org/D57662





More information about the cfe-commits mailing list