[PATCH] CMake+Ninja Linux buildbots
Renato Golin
renato.golin at linaro.org
Thu Sep 4 09:47:45 PDT 2014
Hi David,
Thanks for the review! Comments inline...
On 4 September 2014 16:29, David Blaikie <dblaikie at gmail.com> wrote:
> I'd probably avoid introducing a different ordering of the name
> "checkout_clang_extra_tools" (where the repository is clang_tools_extra, etc
> - so just name it "checkout_clang_tools_extra" for consistency)
Of course!
> The stage two part could maybe benefit from an early return rather than
> indenting the whole thing
Good point, will do.
> Other than that it looks like a totally plausible/good start to me.
Great! I'll commit with those changes and build up from there.
cheers,
--renato
More information about the llvm-commits
mailing list