> On 05/31/2015 01:55 AM, Marek Polacek wrote: > >On Sun, May 31, 2015 at 05:11:52AM +0200, Jan Hubicka wrote: > >>@@ -1548,6 +1503,10 @@ eliminate_unnecessary_stmts (void) > >> something_changed |= remove_dead_phis (bb); > >> } > >> > >>+ if (bb_postorder) > >>+ free (bb_postorder); > >>+ bb_postorder = NULL; > > > >Since free (NULL) is a no-op, that check seems redundant. > Right. I believe Jim M. eliminated all those a couple years ago. > No need to check that it's non-null. Just free it :-)
OK, updated in my local copy of the patch. Honza > > jeff
