Re: [PATCH] Fix up optimize_clobbers (PR tree-optimization/51117)

2011-12-09 Thread Richard Henderson
On 12/09/2011 09:11 AM, Jakub Jelinek wrote: > PR tree-optimization/51117 > * tree-eh.c (optimize_clobbers): Don't remove just one > clobber, but all consecutive clobbers before RESX. > Use gimple_clobber_p predicate. Ok. r~

[PATCH] Fix up optimize_clobbers (PR tree-optimization/51117)

2011-12-09 Thread Jakub Jelinek
Hi! When working on clobber sinking, I've noticed that optimize_clobbers would mistakenly remove just one clobber instead of all consecutive ones, because gsi_remove moves the iterator to the next rather than previous stmt (i.e. GIMPLE_RESX if no debug stmts) and we return in the next iteration.