<div dir="ltr">Thanks, committed as r183794.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 11, 2013 at 11:52 AM, Chad Rosier <span dir="ltr"><<a href="mailto:mcrosier@apple.com" target="_blank">mcrosier@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eric is correct.  Sorry for the belated response.<br>
<span class="HOEnZb"><font color="#888888"><br>
 Chad<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Jun 11, 2013, at 11:50 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
<br>
> I'd assume that Chad's LGTM is still valid after the slight style<br>
> changes. LGTM too.<br>
><br>
> -eric<br>
><br>
> On Mon, Jun 10, 2013 at 5:35 PM, JF Bastien <<a href="mailto:jfb@google.com">jfb@google.com</a>> wrote:<br>
>> Here's an updated patch with the array in an anonymous namespace, with a<br>
>> name on the struct type, and a better description.<br>
>><br>
>> I also fixed i < e to i != e as pointed out off-list by Stephen.<br>
>><br>
>> Chad, Still LGTY?<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>
</div></div></blockquote></div><br></div>