On 05/31/2015 01:55 AM, Marek Polacek wrote:
Right. I believe Jim M. eliminated all those a couple years ago. No need to check that it's non-null. Just free it :-)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.
jeff