ilya-biryukov requested changes to this revision. ilya-biryukov added inline comments. This revision now requires changes to proceed.
================ Comment at: clang-tidy/tool/ClangTidyMain.cpp:345 + if (!Buffer) { + llvm::errs() << diag::err_missing_vfs_overlay_file << OverlayFile; + return; ---------------- This code will print only the enum's integral value, we want to print an actual error message. I don't think there's an easy way to reuse clang's diagnostics here, we should spell out the error message explicitly. ================ Comment at: clang-tidy/tool/ClangTidyMain.cpp:430 + llvm::IntrusiveRefCntPtr<vfs::OverlayFileSystem> BaseFS( + new vfs::OverlayFileSystem(vfs::getRealFileSystem())); ---------------- We should only create an overlay is `-vfsoverlay` was passed and use `getRealFileSystem` without wrappers in the common case. ================ Comment at: clang-tidy/tool/ClangTidyMain.cpp:433 + if (!VfsOverlay.empty()) { + pushVfsOverlayFromFile(VfsOverlay, *BaseFS); + } ---------------- Could we stop clang-tidy with an error if we couldn't create an overlay? That seems like a better option than silently running without an overlay when it was actually specified. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D41535 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits