[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.
> 
> LGTY?

LGTM.

— 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