<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 3, 2015 at 6:18 AM, Sergey Dmitrouk <span dir="ltr"><<a href="mailto:sdmitrouk@accesssoftek.com" target="_blank">sdmitrouk@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Jun 02, 2015 at 03:48:23PM -0700, David Blaikie wrote:<br>
> > Not sure how to handle this yet, a couple of obvious tries causes test<br>
> > failures, will look for something better.<br>
><br>
> Given the debug info size/quality regression here - perhaps we could<br>
> disable this again, temporarily, until we have an answer?<br>
<br>
</span>Looks like we don't need to do this.  I was wrong thinking that the fix<br>
causes tests to fail, there were debugging aid kind of changes left in<br>
the code, which I forgot to revert before running tests.<br>
<br>
Please take a look at the attached patch.<br></blockquote><div><br>Looks good - please commit when ready.<br><br>Optional thoughts you can roll into the commit or we can handle in post commit:<br><br>* The CHECK lines might be a bit more verbose/selectively chosen to be more self documenting (perhaps CHECK for a .loc of line 9, then CHECK-NOT for a .loc, then CHECK for the movl $1 - essentially saying "ensure that the constant 1 is attributed to line 9")<br>* You could drop the "getDebugLoc" from the condition you're adding, since it doesn't matter whether it has one or not if we're just removing it anyway<br>* The comment's english is a bit confusing. Specifically in the last clause of the first sentence. Maybe: "Remove the debug location from the node as the node is about to be used in a location which may differ from the original debug location" ? I'm not sure if that's accurate or not, though.<br><br>Just some thoughts - thanks again!<br><br>- David<br><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
Sergey<br>
</blockquote></div><br></div></div>