This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [lto][patch] Don't compress the IL symbol table
- 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>
- Date: Thu, 1 Oct 2009 09:07:06 -0400
- Subject: Re: [lto][patch] Don't compress the IL symbol table
- References: <38a0d8450909301614p1aee4927q8dcba00d20293343@mail.gmail.com>
On Wed, Sep 30, 2009 at 19:14, Rafael Espindola <espindola@google.com> wrote:
> The plugin doesn't expect the symbol table to be compressed. This
> patch disables compression for the section used to store the IL symbol
> table. It would also be possible to change the plugin so that it could
> read compressed symbol tables, but it is probably better to keep it
> simpler.
Agreed. If size becomes a problem, we can revisit later.
> 2009-09-30 ÂRafael Avila de Espindola Â<espindola@google.com>
>
> Â Â Â Â* lto-opts.c (lto_write_options): Update call to lto_begin_section.
> Â Â Â Â* lto-section-out.c (lto_begin_section): Add a compress argument.
> Update all callers.
> Â Â Â Â(produce_symtab): Don't compress.
> Â Â Â Â* lto-section-out.h (lto_begin_section): Add a compress argument.
Watch for line lengths.
> @@ -168,7 +168,7 @@ static struct lto_compression_stream *compression_stream = NULL;
> /* Begin a new output section named NAME. */
>
> void
> -lto_begin_section (const char *name)
> +lto_begin_section (const char *name, bool compress)
Document new parameter.
OK with that change.
Diego.