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

2017-03-02 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld abandoned this revision. Hahnfeld added a comment. http://lists.llvm.org/pipermail/cfe-dev/2017-January/052512.html suggested to install runtime libraries (like libomp, but I think libc++ in certain cases may fall under the same category) to ResourceDir and I wanted the headers to be c

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

2017-03-02 Thread 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. > > > > > > The

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

2017-03-02 Thread Jonathan Roelofs via Phabricator via cfe-commits
jroelofs added a comment. 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

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

2017-03-02 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld updated this revision to Diff 90296. Hahnfeld edited the summary of this revision. Hahnfeld added a comment. Adapt and add tests. https://reviews.llvm.org/D30214 Files: lib/Driver/ToolChains.cpp test/Driver/Inputs/resource_dir_libcxx/include/c++/v1/.keep test/Driver/Inputs/resour

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

2017-03-01 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added a comment. 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 wi

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

2017-03-01 Thread Jonathan Roelofs via Phabricator via cfe-commits
jroelofs added a comment. libc++ headers should not be installed in the resource dir. https://reviews.llvm.org/D30214 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

2017-03-01 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added a comment. Ping https://reviews.llvm.org/D30214 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

2017-02-21 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld created this revision. https://reviews.llvm.org/D30214 Files: lib/Driver/ToolChains.cpp Index: lib/Driver/ToolChains.cpp === --- lib/Driver/ToolChains.cpp +++ lib/Driver/ToolChains.cpp @@ -4642,6 +4642,7 @@ std::string