[libcxx-commits] [PATCH] D118616: [libc++] Use -I instead of -isystem to include headers in the test suite

[Joe Loser via Phabricator via libcxx-commits]

jloser added a comment.

> Ideally, we'd be able to still use -isystem, but instruct Clang to still emit warnings from the libc++ headers (e.g. we could tell Clang to emit warnings in any file inside <...>/usr/include/c++/v1).

I think that this can be done with `--no-system-header-prefix=path/to/usr/include/c++/v1` FWIW. Note that while Clang supports this, GCC does not for example.

In any case, I like the ideas for this patch; I'm fine with your current solution.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118616