hokein added a subscriber: bkramer. hokein added inline comments.
================ Comment at: include/clang/Tooling/Tooling.h:331 + /// \returns 0 on success; 1 if any error occured; 2 if there is no error but + /// some files are skipped due to missing compile commands. int run(ToolAction *Action); ---------------- I'm not sure whether this new behavior will cause any regression issues (many tools invoke this method like `if (Tool.run(...))`) * before the patch, return 0 if some files are skipped (we think it succeeds) * after the patch, return 2 if some files are skipped (we think it fails) Maybe @bkramer has more words on it? Repository: rC Clang https://reviews.llvm.org/D42361 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
