[cfe-dev] [libcxx] test/lit.cfg patch

Marshall Clow mclow.lists at gmail.com
Tue Jan 28 16:44:13 PST 2014

On Jan 28, 2014, at 3:56 PM, David Fang <fang at csl.cornell.edu> wrote:

> Hi,
> Here's another revision of the patch that doesn't call lit.util.which when cxx_under_test is already !None.  In either case, lit_config.note() still reports the value of cxx_under_test, makes sure user is running the intended clang++ binary.


— Marshall

> David
>> Hi,
>> 	I discovered that when testing libc++ with cmake/lit (make check-libcxx), it was choosing an installed clang++ from my $PATH over the value passed in ${builddir}/test/lit.site.cfg (which is an uninstalled clang++). The attached patch fixes this, and seems to match the comment in context. Ok to commit?
>> David
> -- 
> David Fang
> http://www.csl.cornell.edu/~fang/
> <cxx_under_test.patch>_______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev

More information about the cfe-dev mailing list