[PATCH] D58592: [clang] [ToolChains/NetBSD] Support relative libc++ header path

[Michał Górny via Phabricator via cfe-commits]

mgorny marked an inline comment as done.
mgorny added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/NetBSD.cpp:430
+    // system install from src
+    getDriver().SysRoot + "/usr/include/c++",
+  };
----------------
krytarowski wrote:
> I propose to go for:
> 
> `getDriver().SysRoot + "/usr/include/c++/v1",` with a fallback for `getDriver().SysRoot + "/usr/include/c++",`
> 
> This innocent customization of paths triggers a lot of headache.
> 
> We should switch system location to `/usr/include/c++/v1` for next version of LLVM (9.0 as 8.0 is branched and will be released soon).
> 
> This way we will keep compat with legacy paths and new clang.
This is already handled by the relative path (when `getDriver().Dir` is `/usr/bin`).


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58592/new/

https://reviews.llvm.org/D58592