On Mon, Sep 26, 2011 at 14:42, Jan Hubicka <hubi...@ucw.cz> wrote:
> Hi,
> this patch implements slim LTO. It is updated version of Andi's patch.  It is
> done by terminating IPA optimization after analyzing and outputting LTO and in
> compile_file skipping stuff that outputs assembly.

Nice!  Have you measured object size reduction factors?  Compilation
time differences?


> + @option{-fno-fat-lto-objects} improves compilation time over plain LTO, but
> + requires the complete toolchain to be aware of LTO.
>  Requires a linker with

s/Requires/It requires/

> + linker plugin support for basic functionality.  Nm, ar and ranlib has to be
> + also linker plugin enabled to allow full featured build environment 
> (capable of
> + building static libraries etc).

s/Nm, ar and ranlib has to be also.../Additionally, nm, ar and ranlib
need to support linker plugins to allow a full-featured/


> --- 779,787 ----
>  #else
>        error_at (loc, "LTO support has not been enabled in this 
> configuration");
>  #endif
> !       if (!opts->x_flag_fat_lto_objects && !HAVE_LTO_PLUGIN)
> !         error_at (loc, "-fno-fat-lto-objects are suppored only with linker 
> plugin.");

s/suppored/supported/


OK with those changes.


Diego.

Reply via email to