<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">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><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><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></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>