
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Jul 8, 2011, at 10:28 AM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><p>My current idea is to try to make the "filename " be <token pasting buffer>…</p><div><br></div></blockquote><div><br></div><div>How about maybe folding the token text into the first note ? e.g:</div><div><br></div><div>t.c:3:45: note: pasted token '12' instantiated from:</div><br><blockquote type="cite"><p>(Sent from phone)<br>

</p>

<div class="gmail_quote">On Jul 8, 2011 10:15 AM, "Argyrios Kyrtzidis" <<a href="mailto:kyrtzidis@apple.com">kyrtzidis@apple.com</a>> wrote:<br type="attribution">> On Jul 7, 2011, at 6:09 PM, Chandler Carruth wrote:<br>

> <br>>> On Thu, Jul 7, 2011 at 2:19 PM, Argyrios Kyrtzidis <<a href="mailto:kyrtzidis@apple.com">kyrtzidis@apple.com</a>> wrote:<br>>> Also change the following SM.createInstantiationLoc below this, to handle arguments for the "## __VA_ARGS__" code path<br>

>> <br>>> Done in r134674, please review and let me know if you were thinking of something else.<br>> <br>> Thanks, I mainly wanted the arguments that are not involved in the paste to be handled correctly, as they are now.<br>

> <br>>> <br>>> I've included a few test cases based on yours, but not your exact one. Let me know if there were particular things in your example you wanted to see tested.<br>>> <br>>> There is still a suboptimal source location. I've left a fixme in the test case and am looking at how hard fixing that would be.<br>

> <br>> I also don't like seeing<br>> <br>> <scratch space>:2:1: note: instantiated from:<br>> 12<br>> ^<br>> <br>> seeing a full source location from "scratch space" is not very useful, not sure what's the best way to improve it though..<br>

</div>

</blockquote></div><br></body></html>