llvm dev - Nov 2010 - [LLVMdev] Replacing phi nodes in ScalarEvolution while preserving LCSSA form

Hi Dan, I whipped up this patch as a more optimal test of whether
replacing the phi node PN by V will break LCSSA form.  I hope it
is more or less correct, but since I know nothing about loops it
could well be completely bogus :)  Can you please comment on it.

Best wishes,

Duncan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lcssa.diff
Type: text/x-patch
Size: 1697 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20101117/3639dc6a/attachment.bin>

On Nov 17, 2010, at 11:03 AM, Duncan Sands wrote:
> Hi Dan, I whipped up this patch as a more optimal test of whether
> replacing the phi node PN by V will break LCSSA form.  I hope it
> is more or less correct, but since I know nothing about loops it
> could well be completely bogus :)  Can you please comment on it.
Looks good to me.

Thanks,

Dan

It doesn't seem to fix PR8622.

On Nov 17, 2010, at 8:39 PM, Dan Gohman wrote:
> On Nov 17, 2010, at 11:03 AM, Duncan Sands wrote:
> 
>> Hi Dan, I whipped up this patch as a more optimal test of whether
>> replacing the phi node PN by V will break LCSSA form.  I hope it
>> is more or less correct, but since I know nothing about loops it
>> could well be completely bogus :)  Can you please comment on it.
> 
> Looks good to me.
> 
> Thanks,
> 
> Dan
> 
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev