[PATCH] D65833: [Tooling] Expose ExecutorConcurrency option.

2019-08-07 Thread Diego Astiazarán via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL368196: [Tooling] Expose ExecutorConcurrency option. (authored by DiegoAstiazaran, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D65833: [Tooling] Expose ExecutorConcurrency option.

2019-08-07 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Only hesitation is that AllTUsExecution is one executor of (potentially) many, so this is a weird place for a common flag. However it's already "global" in the sense that --help will

[PATCH] D65833: [Tooling] Expose ExecutorConcurrency option.

2019-08-06 Thread Diego Astiazarán via Phabricator via cfe-commits
DiegoAstiazaran created this revision. DiegoAstiazaran added a reviewer: juliehockett. https://reviews.llvm.org/D65833 Files: clang/include/clang/Tooling/AllTUsExecution.h clang/lib/Tooling/AllTUsExecution.cpp Index: clang/lib/Tooling/AllTUsExecution.cpp