<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 21, 2017 at 4:56 PM Paul Robinson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">probinson accepted this revision.<br>
probinson added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
Well then!  LGTM.<br>
<br>
In <a href="https://reviews.llvm.org/D38088#878342" rel="noreferrer" target="_blank">https://reviews.llvm.org/D38088#878342</a>, @echristo wrote:<br>
<br>
> I mean, I disagree with the general behavior, but I think I've lost that argument so if it's consistent go ahead with it :)<br>
<br>
<br>
I am not super excited about the tactic either, but preserving the line info on instructions that get moved around like this is messing up an important consumer (SPGO) so I'm willing to tolerate it until we can come up with something better.<br></blockquote><div><br>If this is motivated by SPGO issues - should it be using mergeDebugLocations, then (& thus only changing the location if they come from different basic blocks)?<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D38088" rel="noreferrer" target="_blank">https://reviews.llvm.org/D38088</a><br>
<br>
<br>
<br>
</blockquote></div></div>