phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: clang/lib/Driver/ToolChains/Linux.cpp:868 -static std::string DetectLibcxxIncludePath(StringRef base) { +static std::string DetectLibcxxIncludePath(StringRef base, llvm::vfs::FileSystem& vfs) { std::error_code EC; ---------------- Nit: can you make `vfs` the first argument? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64381/new/ https://reviews.llvm.org/D64381 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits