rprichard added a comment.
This change looks like an improvement to me. The NDK currently puts the libc++
headers into the sysroot, but putting it in the usual toolchain include
location seems better.
================
Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:3108
+ SmallString<128> TargetDir(Path);
+ llvm::sys::path::append(TargetDir, Target, "c++", Version);
if (D.getVFS().exists(TargetDir))
----------------
Will the `Target` here have an Android API level suffix?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159292/new/
https://reviews.llvm.org/D159292
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
