[PATCH][Solaris] Use the GCC Installation detector to add the C++ include paths

[Xan López via llvm-commits]

On Wed, Aug 26, 2015 at 10:28:52AM -0400, Rafael Espíndola wrote:
> LGTM

Thanks! Can you commit the patch for me?

Xan

> 
> On 17 August 2015 at 06:16, Xan López <xan at igalia.com> wrote:
> > Hi Yaron, Rafael,
> >
> > thanks, I updated the patch with a test. Please note I'm using the
> > testing ifrastructure already created in a previous patch Rafael
> > reviewed once (that's why the sysroot is not created, among other
> > things).
> >
> > Patch attached.
> >
> > On Wed, Aug 05, 2015 at 04:54:28PM +0300, Yaron Keren wrote:
> >> I made similar test for mingw toolchain in clang/test/Driver/mingw.cpp :
> >> create a directory structure under Inputs, direct clang to it using
> >> --sysroot and test that the expected directories are found.
> >>
> >>
> >>
> >> 2015-08-05 16:15 GMT+03:00 Rafael Espíndola <rafael.espindola at gmail.com>:
> >>
> >> > This looks ok, but can you add a testcase?
> >> >
> >> > On 28 July 2015 at 06:20, Xan López <xan at igalia.com> wrote:
> >> > > A follow up to the previous patch, this gets rid of the last hardcoded
> >> > > path (and moves it to its proper place).
> >> > >
> >> > > _______________________________________________
> >> > > llvm-commits mailing list
> >> > > llvm-commits at cs.uiuc.edu
> >> > > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >> > >
> >> > _______________________________________________
> >> > llvm-commits mailing list
> >> > llvm-commits at lists.llvm.org
> >> > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> >> >