On Wed, Nov 16, 2016 at 11:52:23AM -0700, Jeff Law wrote: > >2016-10-30 Segher Boessenkool <seg...@kernel.crashing.org> > > > > PR rtl-optimization/71785 > > * bb-reorder.c (duplicate_computed_gotos_find_candidates): New > > function, factored out from pass_duplicate_computed_gotos::execute. > > (duplicate_computed_gotos_do_duplicate): Ditto. Don't use > > BB_VISITED. > > (pass_duplicate_computed_gotos::execute): Rewrite. Rerun the pass as > > long as it makes changes. > OK. I'm just going to note for the record here that while we iterate > until nothing changes, the statement and block clamps should in practice > ensure we hit a point where nothing changes.
It is hard limited as well, the loop is not infinite. Anyway, I said I'd revisit this in stage 3; posting the v3 patch now. Segher