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 :-)

jeff

Reply via email to