<div class="gmail_quote">On 3 February 2012 10:02, Chad Rosier <span dir="ltr"><<a href="mailto:mcrosier@apple.com" target="_blank">mcrosier@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 Feb 3, 2012, at 12:45 AM, Duncan Sands <<a href="mailto:baldrick@free.fr" target="_blank">baldrick@free.fr</a>> wrote:<br>
<br>
> On 03/02/12 03:02, Chad Rosier wrote:<br>
>><br>
>> On Feb 2, 2012, at 5:57 PM, Nick Lewycky <<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a><br>
>> <mailto:<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>>> wrote:<br>
>><br>
>>> On 2 February 2012 17:07, Dan Gohman <<a href="mailto:gohman@apple.com" target="_blank">gohman@apple.com</a><br>
>>> <mailto:<a href="mailto:gohman@apple.com" target="_blank">gohman@apple.com</a>>> wrote:<br>
>>><br>
>>>    Author: djg<br>
>>>    Date: Thu Feb 2 19:07:01 2012<br>
>>>    New Revision: 149654<br>
>>><br>
>>>    URL: <a href="http://llvm.org/viewvc/llvm-project?rev=149654&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=149654&view=rev</a><br>
>>>    <<a href="http://llvm.org/viewvc/llvm-project?rev=149654&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=149654&view=rev</a>><br>
>>>    Log:<br>
>>>    Fix SSAUpdaterImpl's RecordMatchingPHI to record exactly the<br>
>>>    PHI nodes which were matched, rather than climbing up the<br>
>>>    original PHI node's operands to rediscover PHI nodes for<br>
>>>    recording, since the PHI nodes found that are not<br>
>>>    necessarily part of the matched set.<br>
>>>    This fixes rdar://10589171.<br>
>>><br>
>>><br>
>>> You mean PR11535?<br>
>><br>
>> Yes, Eli cloned PR11535 to rdar://10589171; the two are synonymous.<br>
><br>
> Except that a rdar reference is useless to people outside of Apple.<br>
><br>
<br>
</div>We do our best to references PRs.  Eli did commented in the PR that it had been cloned to rdar://10589171.<br></blockquote><div><br></div><div>Sorry, I didn't mean to suggest any annoyance. I meant for it to serve as a reminder to close the PR if it's fixed, or else explain what else in the PR is left that wasn't part of this commit (it isn't clear to me that the rdar is synonymous with the PR vesus being one particular issue from the testcase, etc.). I do appreciate the effort to go back to PR numbers, I have to do the same thing myself when going out of our internal bugtracker.</div>


<div><br></div><div>Nick</div><div><br></div></div>