On Jul 2, 2012, at 14:06 , Chandler Carruth <[email protected]> wrote:
> On Mon, Jul 2, 2012 at 2:03 PM, Jordan Rose <[email protected]> wrote:
>
> On Jul 2, 2012, at 13:53 , Chandler Carruth <[email protected]> wrote:
>
> > It was obvious: r159582. I can't promise that's *enough* but it seems
> > definitely a good initial step. Let me know if that helps.
>
> Yes, it does! The only thing left is "Test suite 'Clang-Unit" contained no
> tests". And indeed, ClangUnitTests is not a target in my standalone-build
> project, but it and its subtargets do show up in my combined project.
>
> Ah, ok, I have a patch for that to. It's also pretty obvious/easy when you
> describe it. Give me a moment. =]
>
> That said, the patch will remove the message. We can't build the Clang
> unittests in a standalone build. =/ At least, I'm not particularly interested
> in adding the various build steps necessary for that -- i'd rather make the
> not-standalone build just as fast for IDE users.
…unfortunate, but fair enough. (I see that the add_unittest function does
enough that it's not something we want duplicated in Clang's CMakeLists.txt.)
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
