
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Jul 7, 2011, at 6:09 PM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Thu, Jul 7, 2011 at 2:19 PM, Argyrios Kyrtzidis <span dir="ltr"><<a href="mailto:kyrtzidis@apple.com">kyrtzidis@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div>Also change the following SM.createInstantiationLoc below this, to handle arguments for the "## __VA_ARGS__" code path</div></blockquote></div><br><div>Done in r134674, please review and let me know if you were thinking of something else.</div></blockquote><div><br></div><div>Thanks, I mainly wanted the arguments that are not involved in the paste to be handled correctly, as they are now.</div><br><blockquote type="cite">

<div><br></div><div>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.</div><div><br></div><div>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.</div>

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