<div dir="ltr">Sorry, I missed this because it was on the same thread. I'll look at it now.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 30, 2014 at 5:04 PM, Meador Inge <span dir="ltr"><<a href="mailto:meadori@codesourcery.com" target="_blank">meadori@codesourcery.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping^2.<br>
<div class="HOEnZb"><div class="h5"><br>
On 01/24/2014 01:43 PM, Meador Inge wrote:<br>
> Ping.  You all OK with the general direction of this cleanup?<br>
><br>
> On 01/17/2014 02:59 PM, Meador Inge wrote:<br>
>> Chandler, Hal,<br>
>><br>
>> On 11/16/2013 06:32 PM, Chandler Carruth wrote:<br>
>><br>
>>> On Fri, Nov 15, 2013 at 4:40 PM, Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a><br>
>>> <mailto:<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>>> wrote:<br>
>>><br>
>>>     Chandler, are you okay with me committing this as is? As noted, I think that<br>
>>>     it matches the existing design.<br>
>>><br>
>>><br>
>>> I suppose. This is bad enough that someone needs to go clean up the entire file<br>
>>> immediately... but I guess I shouldn't hold up this patch on that front.<br>
>><br>
>> Many apologies for the delay.  I finally got around to taking a look at what we<br>
>> can do to clean this file up.  I am not looking for a full review now (I will<br>
>> send out a proper separate patch for that), but could y'all give it a quick<br>
>> look to see if the general approach is better?  If the general approach is<br>
>> OK, then I will do another pass to clean things up a bit more.<br>
>><br>
>> I believe I have simplified the design quite a bit and find the result much<br>
>> more straight forward.  All the little classes are gone and I was able to drop<br>
>> about 200 lines of code.<br>
>><br>
>> I am attaching the new file and diff since there is so much noise in the diff.<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>
<br>
<br>
--<br>
Meador Inge<br>
CodeSourcery / Mentor Embedded<br>
</div></div></blockquote></div><br></div>