<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 5, 2013 at 10:17 AM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@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><br>
On Mar 5, 2013, at 10:00 AM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
<br>
> Hi Adrian,<br>
><br>
> Thinking about this I wonder if we might want two EmitLocation routines instead, with one to force the column. Another thought is just to always emit it and ignore it in the backend. Can you take a look at that and see whether it would be better?<br>


<br>
</div>I believe one of the original reasons behind removing column info by default is that getColumnNumber() was deemed too expensive to call every time. This would speak against dropping it in the backend.<br>
see also <a href="http://llvm.org/bugs/show_bug.cgi?id=14106" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14106</a>.<br>
<br></blockquote><div><br></div><div style>Yeah, sorry, brain misfire.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As for created two versions of EmitLocation, I can do that of course, but it seems to be an unnecessary duplication of code to me. What would the benefit be? Is it about the cost of the “hidden" 3rd argument?<br>
<div><br></div></blockquote><div><br></div><div>I have no particular preference either way really.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
<br>
> Also please update the comment to not reference internal apple clang versions and instead reference svn revisions (or, instead and better, just describe the changes).<br>
<br>
</div>done. (I’ll send out a new patch after we reached a consensus about what to best do with emitLocation()).<br><br></blockquote><div><br></div><div style>Sorry for the delay, I'm wondering if there's a way to look at the location in the backend - even without the column information to make a determination here. I'll probably look into it before putting this in though.</div>
<div style><br></div><div style>-eric</div><div style><br></div></div></div></div>