Replacing phi nodes in ScalarEvolution while preserving LCSSA form

Project Infrastructure LLVM Dev List Archives
#1

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 :slight_smile: Can you please comment on it.

Best wishes,

Duncan.

lcssa.diff (1.66 KB)

#2

Looks good to me.

Thanks,

Dan

#3

It doesn't seem to fix PR8622.

#4

Hi Jakub,

It doesn't seem to fix PR8622.

I wasn't aware of PR8622 - I will take a look.

Ciao,

Duncan.