Right, sorry, I didn't mean that this would be predicated on a 'release' build (indeed lots of people work with release+asserts builds for speed & I think they still run the regression tests that need those tools)
Perhaps to rephrase: could there be a target that builds the same that are installed by the 'install' target (& if it's depending on test tools, that seems like a bug)? Seems like a reasonable target to have. From: Bill Wendling Sent: 9/27/2012 9:21 AM To: David Blaikie Cc: [email protected] Subject: Re: [cfe-commits] r164739 - /cfe/trunk/tools/Makefile I'm not sure. I could imagine them wanting release builds of the testing stuff, so I didn't want to mess with that...But I can be convinced otherwise. :) -bw On Sep 26, 2012, at 11:08 PM, David Blaikie <[email protected]> wrote: > I don't know, but would it make more or less sense to just have a build > target that just builds the release binaries, rather than a variable? > From: Bill Wendling > Sent: 9/27/2012 3:14 AM > To: [email protected] > Subject: [cfe-commits] r164739 - /cfe/trunk/tools/Makefile > Author: void > Date: Wed Sep 26 19:11:09 2012 > New Revision: 164739 > > URL: http://llvm.org/viewvc/llvm-project?rev=164739&view=rev > Log: > Use a variable to stop us from building clang testing tools. > <rdar://problem/11202465> > > Modified: > cfe/trunk/tools/Makefile > > Modified: cfe/trunk/tools/Makefile > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/Makefile?rev=164739&r1=164738&r2=164739&view=diff > ============================================================================== > --- cfe/trunk/tools/Makefile (original) > +++ cfe/trunk/tools/Makefile Wed Sep 26 19:11:09 2012 > @@ -14,6 +14,10 @@ > DIRS := driver libclang c-index-test arcmt-test c-arcmt-test diagtool \ > clang-check > > +ifeq ($(CLANG_NO_TEST_TOOLS),YES) > +DIRS := driver libclang > +endif > + > # Recurse into the extra repository of tools if present. > OPTIONAL_DIRS := extra > > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
