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

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 5 06:55:49 PST 2018

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 push_back` would be a line shorter.

Comment at: lib/Tooling/AllTUsExecution.cpp:125
     llvm::Regex RegexFilter(Filter);
     for (std::string File : Files) {
This is unused now

  rC Clang


More information about the cfe-commits mailing list