<div dir="ltr">Look good. Do you have commit access or should I commit this for you?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 16, 2014 at 5:55 PM, Eugene Toder <span dir="ltr"><<a href="mailto:eltoder@gmail.com" target="_blank">eltoder@gmail.com</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">Makes sense. Moved into unittests.</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 16, 2014 at 11:46 AM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</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">Thanks. However, tests like these should go in unittests/Format/FormatTest.cpp. Sorry for not being more precise before.</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 16, 2014 at 5:41 PM, Eugene Toder <span dir="ltr"><<a href="mailto:eltoder@gmail.com" target="_blank">eltoder@gmail.com</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">Thank you. Now with a test.<span><font color="#888888"><div><br></div><div>Eugene</div></font></span></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 15, 2014 at 6:10 PM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</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">I think the change itself look good (although I don't use inline assembly much). Please add a test.</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Mon, Sep 15, 2014 at 11:14 PM, Eugene Toder <span dir="ltr"><<a href="mailto:eltoder@gmail.com" target="_blank">eltoder@gmail.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Hello,<div><br></div><div>When inline assembly has only clobber constraints it's common to write all three colons without spaces, for example:</div><div><br></div><div>asm volatile("nop" ::: "memory");</div><div><br></div><div>Currently clang-format splits ::: into coloncolon and colon, and inserts a space before the colon, which looks strange:</div><div><div><br>asm volatile("nop" :: : "memory");</div></div><div><br></div><div>I suggest to allow having ::: without spaces in this case. Attached is a simple patch to that effect.</div><div><br></div><div>Thanks,</div><div>Eugene</div></div>
<br></div></div>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>