Re: [PATCH] Fix endless loop in forwprop (PR tree-optimization/53226)

2012-05-09 Thread Jakub Jelinek
On Wed, May 09, 2012 at 10:21:04AM +0200, Richard Guenther wrote: > Using the stmt UID looks odd as you are using it as flag - why not use > one of the two stmt flags available to passes? > > Btw, the other possibility would be to simply change the various combiners > to require them to update the

Re: [PATCH] Fix endless loop in forwprop (PR tree-optimization/53226)

2012-05-09 Thread Richard Guenther
On Tue, May 8, 2012 at 8:33 PM, Jakub Jelinek wrote: > Hi! > > The attached testcase loops endlessly, using more and more memory. > The problem is that the prev stmt iterator sometimes references stmts that > remove_prop_source_from_use decides to remove, and since Michael's > gimple seq changes t

[PATCH] Fix endless loop in forwprop (PR tree-optimization/53226)

2012-05-08 Thread Jakub Jelinek
Hi! The attached testcase loops endlessly, using more and more memory. The problem is that the prev stmt iterator sometimes references stmts that remove_prop_source_from_use decides to remove, and since Michael's gimple seq changes that seems to be fatal. Fixed by not keeping an iterator, but ins