[PATCH] D159292: [driver] Conditionally include installed libc++ headers for Android

Shoaib Meenai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 7 10:58:21 PDT 2023


smeenai added a comment.

In D159292#4632389 <https://reviews.llvm.org/D159292#4632389>, @rprichard wrote:

> 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.

Are you comfortable accepting this, or should I ping other reviewers? :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159292



More information about the cfe-commits mailing list