[llvm-commits] [llvm] r149654 - in /llvm/trunk: include/llvm/Transforms/Utils/SSAUpdaterImpl.h test/Transforms/ScalarRepl/phi-cycle.ll

Chad Rosier mcrosier at apple.com
Fri Feb 3 10:02:40 PST 2012


On Feb 3, 2012, at 12:45 AM, Duncan Sands <baldrick at free.fr> wrote:

> On 03/02/12 03:02, Chad Rosier wrote:
>> 
>> On Feb 2, 2012, at 5:57 PM, Nick Lewycky <nlewycky at google.com
>> <mailto:nlewycky at google.com>> wrote:
>> 
>>> On 2 February 2012 17:07, Dan Gohman <gohman at apple.com
>>> <mailto:gohman at apple.com>> wrote:
>>> 
>>>    Author: djg
>>>    Date: Thu Feb 2 19:07:01 2012
>>>    New Revision: 149654
>>> 
>>>    URL: http://llvm.org/viewvc/llvm-project?rev=149654&view=rev
>>>    <http://llvm.org/viewvc/llvm-project?rev=149654&view=rev>
>>>    Log:
>>>    Fix SSAUpdaterImpl's RecordMatchingPHI to record exactly the
>>>    PHI nodes which were matched, rather than climbing up the
>>>    original PHI node's operands to rediscover PHI nodes for
>>>    recording, since the PHI nodes found that are not
>>>    necessarily part of the matched set.
>>>    This fixes rdar://10589171.
>>> 
>>> 
>>> You mean PR11535?
>> 
>> Yes, Eli cloned PR11535 to rdar://10589171; the two are synonymous.
> 
> Except that a rdar reference is useless to people outside of Apple.
> 

We do our best to references PRs.  Eli did commented in the PR that it had been cloned to rdar://10589171.

 Chad

> Ciao, Duncan.
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list