[PATCH] D64381: Update libc++ include path detection to use VFS on Linux

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 8 17:24:36 PDT 2019


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





More information about the cfe-commits mailing list