Re: Fix phi backedge detection in backprop (PR85989)

2018-06-01 Thread Richard Biener
On Fri, Jun 1, 2018 at 2:38 PM Richard Sandiford wrote: > > This PR is a nasty wrong code bug due to my fluffing a test for a > backedge in gimple-ssa-backprop.c. Backedges are supposed to be > from definitions in the statement we're visiting to uses in statements > that we haven't visited yet.

Fix phi backedge detection in backprop (PR85989)

2018-06-01 Thread Richard Sandiford
This PR is a nasty wrong code bug due to my fluffing a test for a backedge in gimple-ssa-backprop.c. Backedges are supposed to be from definitions in the statement we're visiting to uses in statements that we haven't visited yet. However, the check failed to account for PHIs in the current block