<div dir="ltr">Hi Adrian,<div><br></div><div>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? 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).</div>
<div><br></div><div style>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 1, 2013 at 5:53 PM, 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">On Wed Feb 27 12:45:37 Manman Ren wrote:<br>
> Given an example with two function calls in a single line, without -gcolumn-info, both function calls will be at the same source location.<br>
> And Clang will generate a single coalesced inlining scope since we unique scopes by the source location.<br>
><br>
> Even without -gcolumn-info, we should generate two inlining scopes. This means we should differentiate the different call sites with the same source location.<br>
> One option is to use the column info in SourceLoc to store the call site id when -gcolumn-info is not on.<br>
><br>
> Any problem with this? Any other better suggestions?<br>
<br>
Here’s a patch for review that implements this. I’m using the actual column info rather than the (arbitrary?) call site id.<br>
<br>
thanks,<br>
Adrian<br>
<br>
---------<br>
<br>
Force column info to be generated for call expressions so we<br>
can differentiate multiple call sites on the same line in the<br>
debug info.<br>
<br>
Fixes rdar://problem/13036237<br>
<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>