This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Fix Ada bootstrap with LTO

From: Jeff Law <law at redhat dot com>
To: Marek Polacek <polacek at redhat dot com>, Jan Hubicka <hubicka at ucw dot cz>
Cc: gcc-patches at gcc dot gnu dot org, rguenther at suse dot de
Date: Mon, 01 Jun 2015 09:21:46 -0600
Subject: Re: Fix Ada bootstrap with LTO
Authentication-results: sourceware.org; auth=none
References: <20150531031152 dot GB34108 at kam dot mff dot cuni dot cz> <20150531075539 dot GJ27320 at redhat dot com>

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. Noneed to check that it's non-null. Just free it :-)


jeff

Follow-Ups:
- Re: Fix Ada bootstrap with LTO
  - From: Jan Hubicka

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]