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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to