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

Best wishes,

Duncan.

lcssa.diff (1.66 KB)

Looks good to me.

Thanks,

Dan

It doesn't seem to fix PR8622.

Hi Jakub,

It doesn't seem to fix PR8622.

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

Ciao,

Duncan.