<div dir="ltr">Thank you for reviewing. Made it to store the location of the trigraph or whitespace. New patch is attached. You can also review the same patch at <a href="http://llvm-reviews.chandlerc.com/D777">http://llvm-reviews.chandlerc.com/D777</a>.</div>

<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 9, 2013 at 10:33 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This mostly looks fine, but please store the location of the trigraph or whitespace-after-backslash so that you can give a more accurately-positioned diagnostic. You can use expected-warning@OFFSET to expect a warning on a different line.<div class="HOEnZb">

<div class="h5"><div>
<br><div class="gmail_quote">On Wed, May 8, 2013 at 9:40 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@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">Ping?</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 16, 2013 at 5:35 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@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">There are lots of other folks who can take a look, including Richard, Doug, Argiris, etc... Unfortunately some of us are tied up in C++ standard committee meetings, but there should be replies before too long...</div>





<div class="gmail_extra"><br><br><div class="gmail_quote"><div><div>On Tue, Apr 16, 2013 at 5:28 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@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>On 16 April 2013 04:00, Rui Ueyama <<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>> wrote:<br>






> Yes, I think the new while-loop is not in a very hot path, as it is executed<br>
> only when there's a '/' at the very beginning of a line in a block comment.<br>
> And it loops only when previous lines are continuation lines. That should be<br>
> pretty unusual. I think that's why this minor bug was not found until now :)<br>
><br>
> Can anyone else take a look?<br>
<br>
</div>Chris I guess. He is not on code owners for it must I think most of<br>
the lexer is still his.<br>
<br>
Cheers,<br>
Rafael<br>
</div></div><div><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>
</div></div></div></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<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>
</div></div></blockquote></div><br></div>