<div dir="ltr">Thanks! Committed in r244293 and r244294.<div><br></div><div>-- Sean Silva</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 6, 2015 at 4:52 AM, Débora Setton Sanches <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5"><div>On Wed, Aug 5, 2015 at 4:01 PM, David Wiberg <span dir="ltr"><<a href="mailto:dwiberg@gmail.com" target="_blank">dwiberg@gmail.com</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>2015-08-05 15:30 GMT+02:00 Débora Setton Sanches <<a href="mailto:setton.debora@gmail.com" target="_blank">setton.debora@gmail.com</a>>:<br>
> Hi,<br>
><br>
> While I was reading the IR documentation the other day, I took note of a few<br>
> small typos and grouped them in the patch that is attached.<br>
><br>
> It's really simple stuff, but I think they improve the text nonetheless,<br>
> so... Here you go! :)<br>
><br>
> Thanks,<br>
><br>
> Débora.<br>
><br>
<br>
</div></div>Hi Débora,<br>
<br>
The changes look good to me. I noticed a couple of extra white spaces<br>
between words while reading the patch. Perhaps you can fix it before<br>
this is committed, see below.<br>
<br>
<start patch snippets><br>
+value in the metadata node. This is analogous to the ''dereferenceable''<br>
+attribute on parameters and return values.  This metadata can only be applied<br>
<br>
+value in the metadata node. This is analogous to the<br>
''dereferenceable_or_null''<br>
+attribute on parameters and return values.  This metadata can only be applied<br>
<end patch snippets><br>
<br>
There are some more of the same kind in the file but these are the<br>
ones on lines which you've changed.<br>
<br>
Best regards<br>
David<br>
<br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
><br>
</blockquote></div><br></div><div class="gmail_extra"><br></div></div></div><div class="gmail_extra">Hi David,<div><br></div><div>Thanks for reviewing this!</div><div><br></div><div>I hadn't noticed the double whitespace issue. After you mentioned that, I fixed the original patch (see v2 attached), and I also wanted to send a separate patch fixing only this specific issue for the whole file.</div><div><br></div><div>Since I didn't know how to do that with svn (since svn doesn't have the concept of "local commits"), the second patch is a `git diff` instead of an `svn diff`, is that ok? How should I have done that with svn?</div><div><br></div><div>Thanks,</div><div><br></div><div>Débora.</div><div><br></div><div><br></div><div><br></div></div></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>