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 Repository: rC Clang https://reviews.llvm.org/D54104 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits