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: [lto][patch] Fix reading of cgraph when multiple files are passed to lto1

From: "Diego Novillo" <dnovillo at google dot com>
To: "Rafael Espindola" <espindola at google dot com>
Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, "Simon Baldwin" <simonb at google dot com>
Date: Fri, 24 Oct 2008 10:53:18 -0400
Subject: Re: [lto][patch] Fix reading of cgraph when multiple files are passed to lto1
References: <38a0d8450810240409s68c2540bo4c40345275ae7c8f@mail.gmail.com> <b798aad50810240619h5b6564d6l98554a3f68a14b70@mail.gmail.com> <38a0d8450810240632l584def8fo6b0c0e5d31eac639@mail.gmail.com>

On Fri, Oct 24, 2008 at 09:32, Rafael Espindola <espindola@google.com> wrote:

> Is the attached patch OK if it has no regressions or should I fix the
> cgraph update and commit the two together?

I'd rather do it in two patches.  This is OK if testing succeeds

>    VEC_free (cgraph_node_ptr, heap, nodes);
>    VEC_free (cgraph_node_ptr, heap, del_list);

del_list is not needed anymore.

OK with that change and a CL entry.

Thanks.  Diego.

References:
- [lto][patch] Fix reading of cgraph when multiple files are passed to lto1
  - From: Rafael Espindola
- Re: [lto][patch] Fix reading of cgraph when multiple files are passed to lto1
  - From: Diego Novillo
- Re: [lto][patch] Fix reading of cgraph when multiple files are passed to lto1
  - From: Rafael Espindola

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