Gating the code doesn't actually hide the perf problem. Because the perf problem occurs in getPresumedLoc which calculates line and column number. The debug code was calling getPresumedLoc twice once to only get the line and once to only get the column, but this results in line and column actually being calculated twice.<br>
<br>Perhaps the debug code should be refactored to only call getPresumedLoc once?<br><br><div class="gmail_quote">On Thu, Oct 18, 2012 at 3:05 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">><br>
> Hrm, I skimmed over the "no known current users". I hate such configuration flags, but this one makes sense.<br>
><br>
<br>
Agreed. I just thought it was a waste to remove the code completely<br>
since someone may want the information.<br>
<div class="HOEnZb"><div class="h5"><br>
-eric<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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>