[PATCH] D30214: [Driver] Search for libc++ headers in ResourceDir

[Eric Fiselier via Phabricator via cfe-commits]

EricWF added a comment.

In https://reviews.llvm.org/D30214#690774, @jroelofs wrote:

> In https://reviews.llvm.org/D30214#690375, @Hahnfeld wrote:
>
> > In https://reviews.llvm.org/D30214#690010, @jroelofs wrote:
> >
> > > libc++ headers should not be installed in the resource dir.
> >
> >
> > They are currently not by default. But with https://reviews.llvm.org/D30015 for runtime libraries, this may become a place for headers packaged with the compiler.
>
>
> Note that I didn't say "are not", but rather "should not".
>
> The only things that should live in the resource dir are things that are tied to the specific version of the compiler, i.e. sanitizer runtimes, builtins, and the few compiler-provided headers. Everything else should live in sysroots/SDKs, and the resource dir is not where those should live.


+1. I don't think this is an appropriate place for the headers to live.


https://reviews.llvm.org/D30214