<div dir="ltr">Yes, I think the new while-loop is not in a very hot path, as it is executed only when there's a '/' at the very beginning of a line in a block comment. And it loops only when previous lines are continuation lines. That should be pretty unusual. I think that's why this minor bug was not found until now :)<div>

<br></div><div style>Can anyone else take a look?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 16, 2013 at 4:13 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Bump<br>
<br>
This looks pretty reasonable to me - but the lexer is a pretty hot<br>
path (though this doesn't seem like it would be very hot itself) so<br>
I'm not quite willing to sign off myself, but hopefully this'll be<br>
brought to someone else's attention at some point<br>
<div><div class="h5"><br>
On Mon, Apr 8, 2013 at 7:16 PM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> Hi,<br>
><br>
> This is my first patch to Clang, and this fixes a bug in block comment<br>
> parser (<a href="http://llvm.org/bugs/show_bug.cgi?id=15706" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=15706</a>). After submitting this<br>
> patch, Clang will be able to parse the following block comment.<br>
><br>
> /* *\<br>
> \<br>
> /<br>
><br>
> Currently Clang can skip only one escaped newline between * and /.<br>
><br>
> Rui<br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">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>