[PATCH] D54104: [Tooling] Correct the total number of files being processed when `filter` is provided.

2018-11-05 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL346135: [Tooling] Correct the total number of files being processed when `filter` is… (authored by hokein, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM

[PATCH] D54104: [Tooling] Correct the total number of files being processed when `filter` is provided.

2018-11-05 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 172585. hokein marked 2 inline comments as done. hokein added a comment. Address comments. Repository: rC Clang https://reviews.llvm.org/D54104 Files: lib/Tooling/AllTUsExecution.cpp Index: lib/Tooling/AllTUsExecution.cpp

[PATCH] D54104: [Tooling] Correct the total number of files being processed when `filter` is provided.

2018-11-05 Thread Eric Liu via Phabricator via cfe-commits
ioeric accepted this revision. ioeric added inline comments. This revision is now accepted and ready to land. Comment at: lib/Tooling/AllTUsExecution.cpp:103 +if (!RegexFilter.match(File)) + continue; +Files.push_back(File); nit: `if (match) then

[PATCH] D54104: [Tooling] Correct the total number of files being processed when `filter` is provided.

2018-11-05 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: ioeric. Repository: rC Clang https://reviews.llvm.org/D54104 Files: lib/Tooling/AllTUsExecution.cpp Index: lib/Tooling/AllTUsExecution.cpp === ---