<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 30, 2014 at 2:57 PM, Iain Sandoe <span dir="ltr"><<a href="mailto:iain@codesourcery.com" target="_blank">iain@codesourcery.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
On 30 May 2014, at 21:52, Eric Christopher wrote:<br>
<br>
> On Fri, May 30, 2014 at 1:49 PM, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On Thu, May 29, 2014 at 1:51 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> Yeah I figured Iain would be doing the actual maintenance here.<br>
>><br>
>><br>
>> Unless this is going to be a "developers ignore the Makefiles and Iain<br>
>> cleans up any breaks" situation (which I don't think is very desirable),<br>
>> developers will still need to make some modifications to the Makefiles. I<br>
>> don't mind this too much, but could Iain please explicitly give instructions<br>
>> for how the Makefiles need to be changed when the following relatively<br>
>> 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>
>> (well, unless the answer is "exactly the same as for the core llvm tree")<br>
><br>
> This is what I'd have expected here.<br>
<br>
</div>the design is modelled exactly one what's done for clang et. al. (i.e. I followed the design used for these other modules)<br>
<br>
so, the statement is "you should not notice anything different from what you've already seen".<br>
<br>
If something is not clear, of course, I would do my best to elaborate.<br></blockquote><div><br></div><div>Cool. Sounds good.<br><br></div><div>-- Sean Silva<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888">Iain<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> -eric<br>
><br>
>><br>
>> -- Sean Silva<br>
>><br>
>>><br>
>>> On May 29, 2014 12:45 PM, "Michael Spencer" <<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>> wrote:<br>
>>>><br>
>>>> 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">echristo@gmail.com</a>><br>
>>>> 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">iain@codesourcery.com</a>><br>
>>>>> wrote:<br>
>>>>>> Hi Eric,<br>
>>>>>><br>
>>>>>> sorry this took so long, (other things just kept creeping up the<br>
>>>>>> TODO).<br>
>>>>>><br>
>>>>>> This is a set of Makefiles based on the approach(es) used elsewhere in<br>
>>>>>> the tree.<br>
>>>>>> I've been using these for quite a while on both OSX and Linux with<br>
>>>>>> either clang or GCC as the bootstrap compiler.<br>
>>>>>><br>
>>>>>> With this patch, lld will build (with autoconf & make) when it is<br>
>>>>>> 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.<br>
>>>>>> there's no way to switch lld on/off other than directory presence/absence in<br>
>>>>>> 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>
>>><br>
>>><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>
>><br>
<br>
</div></div></blockquote></div><br></div></div>