<br><br>On Wednesday, May 28, 2014, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
On 29/05/2014 03:44, Eric Christopher wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, May 28, 2014 at 5:41 PM, Alp Toker <<a>alp@nuanti.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Iain,<br>
<br>
I noticed your Makefile updates including this one and lld...<br>
<br>
<br>
<br>
On 28/05/2014 23:25, Iain Sandoe wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
Is there some reason that the unitttests are not run for<br>
clang/tools/extra?<br>
</blockquote>
<br>
The makefile build systems have felt moderately deprecated for some time.<br>
There are odds and ends that are only available in the CMake build system,<br>
particularly tests/examples/documentation -- mostly unnoticed, so we can<br>
infer that developers/users have mostly switched by now.<br>
<br>
Do we really want to begin adding new features to the makefiles again at<br>
this point -- is there something it can do that the primary one can't?<br>
<br>
CMake lets us target multiple platforms equally, generating fast parallel<br>
builds with ninja and letting us drop the GNU make dependency for a lighter<br>
build environment.<br>
<br>
If there's any remaining limitation it seems more appropriate to address<br>
those and keep moving gently towards a single build system.<br>
<br>
</blockquote>
Sadly there are currently enough limitations that it's worth at least<br>
making sure that basic functionality is shared if possible.<br>
</blockquote>
<br>
Are we tracking those limitations somewhere?<br>
<br></blockquote><div><br></div>Pretty sure yes. There's a pr for it I think. I don't have the number handy.<div><br></div><div>-eric<br><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Alp.<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-eric<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Alp.<br>
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
if not, then the following trivial patch makes them run (they do so<br>
without error on osx 10.8).<br>
<br>
Iain<br>
<br>
diff --git a/llvm/test/Makefile b/llvm/test/Makefile<br>
index dc99fe1..20ebb78 100644<br>
--- a/llvm/test/Makefile<br>
+++ b/llvm/test/Makefile<br>
@@ -57,7 +57,7 @@ LIT_ALL_TESTSUITES +=<br>
$(PROJ_OBJ_DIR)/../tools/<u></u>clang/tools/extra/test<br>
     # Force creation of Clang Tools' lit.site.cfg.<br>
   clang-tools-site-cfg: FORCE<br>
-       $(MAKE) -C $(PROJ_OBJ_DIR)/../tools/<u></u>clang/tools/extra/test<br>
lit.site.cfg<br>
+       $(MAKE) -C $(PROJ_OBJ_DIR)/../tools/<u></u>clang/tools/extra/test<br>
lit.site.cfg Unit/lit.site.cfg<br>
   extra-site-cfgs:: clang-tools-site-cfg<br>
   endif<br>
<br>
<br>
______________________________<u></u>_________________<br>
cfe-commits mailing list<br>
<a>cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><br>
</blockquote>
<br>
--<br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</blockquote></blockquote>
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</blockquote></div>