Hello!

> > > Test         O2 ref     patched    Ofast + LTO ref   patched
> > > 164.gzip        12      0 (-100%)            39      0 (-100%)
> > > 175.vpr         0       0 (-0%)              4       0 (-100%)
> > > 176.gcc         141     6 (-96%)             294     10 (-97%)
> > > 181.mcf         4       0 (-100%)            4       2 (-50%)

Do you also have executable sizes at hand?

> 2015-03-10  Ilya Enkovich  <ilya.enkov...@intel.com>
>
> PR target/65103
> * config/i386/i386.c (ix86_address_cost): Fix cost of a PIC
> register.
>
> gcc/testsuite/
>
> 2015-03-10  Ilya Enkovich  <ilya.enkov...@intel.com>
>
> PR target/65103
> * gcc.target/i386/pr65103-1.c: New.

LGTM, just a nit below.

Otherwise, OK for mainline as a bugfix (but please wait for a day if
there are any objections from release managers).

+  /* Attempt to minimize number of registers in the address.

This is now a displaced comment. Please integrate it in the main comment.

Thanks,
Uros.

Reply via email to