[lto][patch] Don't compress the IL symbol table

Diego Novillo dnovillo@google.com
Thu Oct 1 13:07:00 GMT 2009


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.



More information about the Gcc-patches mailing list