<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 29, 2014 at 1:51 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><p dir="ltr">Yeah I figured Iain would be doing the actual maintenance here. </p></blockquote><div><br>
Unless this is going to be a "developers ignore the Makefiles and Iain cleans up any breaks" situation (which I don't think is very desirable), developers will still need to make some modifications to the Makefiles. I don't mind this too much, but could Iain please explicitly give instructions for how the Makefiles need to be changed when the following relatively common development operations need to be done:<br>
<br>- Adding a new file to one of the libraries<br>- Adding a new unittest<br>- Adding a new library (this also includes splitting one library into two)<br>- Modifying the dependencies between libraries.<br><br></div><div>
(well, unless the answer is "exactly the same as for the core llvm tree")<br></div><div><br></div><div>-- Sean Silva<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class=""><div class="h5">
<div class="gmail_quote">On May 29, 2014 12:45 PM, "Michael Spencer" <<a href="mailto:bigcheesegs@gmail.com" target="_blank">bigcheesegs@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

lgtm as long as I don't have to maintain it. There is going to be some<br>
restructuring of the build soonish to fix the circular dependency<br>
problem, if you can update these when that happens then this is fine.<br>
<br>
- Michael Spencer<br>
<br>
<br>
On Thu, May 29, 2014 at 11:34 AM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
> Seems reasonable to me.<br>
><br>
> Michael?<br>
><br>
> -eric<br>
><br>
> On Wed, May 28, 2014 at 8:08 AM, Iain Sandoe <<a href="mailto:iain@codesourcery.com" target="_blank">iain@codesourcery.com</a>> wrote:<br>
>> Hi Eric,<br>
>><br>
>> sorry this took so long, (other things just kept creeping up the TODO).<br>
>><br>
>> This is a set of Makefiles based on the approach(es) used elsewhere in the tree.<br>
>> I've been using these for quite a while on both OSX and Linux with either clang or GCC as the bootstrap compiler.<br>
>><br>
>> With this patch, lld will build (with autoconf & make) when it is found in llvm/tools (using the same approach as lldb).<br>
>><br>
>> Right now, as with lldb, there's no more finesse than that (i.e. there's no way to switch lld on/off other than directory presence/absence in tools/).<br>
>><br>
>> The lld tests are added to check-all.<br>
>><br>
>> does this seem like a reasonable starting point?<br>
>> cheers<br>
>> Iain<br>
>><br>
>><br>
>><br>
>><br>
>><br>
</blockquote></div>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>