On Wed, Jan 27, 2016 at 12:48:31PM -0700, Jeff Law wrote: > On 01/27/2016 12:39 PM, Bernhard Reutner-Fischer wrote: > >On January 27, 2016 8:20:47 PM GMT+01:00, Jeff Law <l...@redhat.com> wrote: > > > >> > >>This also fixes 3 minor issues Bernd spotted in the last round of > >>changes. > > > >Bernhard, not Bernd i suppose. Note that the !is_gimple_debug (stmt) should > >now be redundant in the hunk below. > >Thanks, > Terribly sorry. Yes, it should have been Bernhard. > > The is_gimple_debug is not redundant -- the first statement in the block > could be a debug statement. If we had a gsi-first-non-debug-non-label > function, we could use that and then the is_gimple_debug would be redundant
But then using gsi_next_nondebug doesn't make much sense. Jakub