<div dir="ltr">Nadav: Thanks, nice catches, lack of a fixed width font in gmail when this opened makes it difficult. :)<div><br></div><div>Chris: You can use clang-format to format diffs as well. <a href="http://clang.llvm.org/docs/ClangFormat.html">http://clang.llvm.org/docs/ClangFormat.html</a></div>
<div><br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 1, 2013 at 4:25 PM, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">The indentation is inconsistent (mixed tabs and spaces?) and there are line breaks in unexpected places (after the last return). <div>
<div class="h5"><div>  <br><div><div>On Nov 1, 2013, at 3:06 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:</div><br><blockquote type="cite"><div dir="ltr">Looks pretty reasonable, needs a testcase :)<div>
<br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 31, 2013 at 11:49 AM, Chris Smowton <span dir="ltr"><<a href="mailto:chris@smowton.net" target="_blank">chris@smowton.net</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">See <a href="http://llvm.org/bugs/show_bug.cgi?id=17757" target="_blank">http://llvm.org/bugs/show_bug.<u></u>cgi?id=17757</a> for details; Chris Lattner advised I should cross-post the patch here.<span><font color="#888888"><br>


<br>
Chris<br>
</font></span><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br></div>