[PATCH] D12382: Extend linux header search to find libc++ headers in c++/vN for any N.

Jonathan Roelofs via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 26 15:26:17 PDT 2015


jroelofs added a subscriber: jroelofs.

================
Comment at: lib/Driver/ToolChains.cpp:3535
@@ -3534,1 +3534,3 @@
 
+static std::string DetectLibcxxIncludePath(const std::string &base) {
+  std::error_code EC;
----------------
StringRef instead?

================
Comment at: lib/Driver/ToolChains.cpp:3564
@@ -3544,3 +3563,3 @@
         // The primary location is within the Clang installation.
         // FIXME: We shouldn't hard code 'v1' here to make Clang future proof to
         // newer ABI versions.
----------------
Kill the FIXME while you're here fixing what the FIXME complains about.


Repository:
  rL LLVM

http://reviews.llvm.org/D12382





More information about the cfe-commits mailing list