Duncan Sands
2010-Nov-17 19:03 UTC
[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>
Dan Gohman
2010-Nov-17 19:39 UTC
[LLVMdev] Replacing phi nodes in ScalarEvolution while preserving LCSSA form
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
Jakub Staszak
2010-Nov-18 13:23 UTC
[LLVMdev] Replacing phi nodes in ScalarEvolution while preserving LCSSA form
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
Possibly Parallel Threads
- [LLVMdev] Replacing phi nodes in ScalarEvolution while preserving LCSSA form
- [LLVMdev] Replacing phi nodes in ScalarEvolution while preserving LCSSA form
- [LLVMdev] [RFC] LCSSA vs. SSAUpdater ... FIGHT!
- LSR tests failing under new pass manager due to not being in LCSSA form
- LCSSA verification for the top-level loops