<div dir="ltr">Thanks Reid, committed as r235186.<div><br></div><div>I've also marked <a href="http://llvm.org/pr23071">http://llvm.org/pr23071</a> as fixed.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 14 April 2015 at 19:55, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@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"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Mon, Apr 13, 2015 at 4:17 PM, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.com</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi Reid,<div><br></div><span class=""><div>I was going to pick your brain on this in person but it seems you're not over in London for the conference. Anyhow, I've tried a few variations but I've been unable to find a way to better handle this edge case without doing some heavy refactoring.</div><div><br></div><div>Do you have fundamental objections to the approach taken (and I admit it's not pretty) or is it simply that the comment leaves something to be desired - such as a clearer comment?</div></span></div></blockquote><div><br></div><div>No, not really. Looks like we do this kind of thing elsewhere in the lexer:</div><div><div>    // If we found the stringify operator, get the argument stringified.  The</div><div>    // preprocessor already verified that the following token is a macro name</div><div>    // when the #define was parsed.</div><div>    const Token &CurTok = Tokens[i];</div><div>    if (i != 0 && !Tokens[i-1].is(tok::hashhash) && CurTok.hasLeadingSpace())</div><div>      NextTokGetsSpace = true;</div></div><div><br></div><div>lgtm</div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><span style="background-color:rgb(255,255,255);color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif"><b>Indefiant </b>: <a href="http://www.indefiant.com" target="_blank">http://www.indefiant.com</a></span></div><div><span style="font-family:Arial,Helvetica,sans-serif"><font color="#999999">Home of Recode : Runtime C++ Editing for VS</font></span></div></div></div>
</div>