On Thu, Mar 12, 2015 at 10:50 AM, Ilya Enkovich <enkovich....@gmail.com> wrote:
>> > > > 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? > > Summary size change for SPEC2000 on -O2 is -0,11%. Nice! >> > 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. > > Here is a final version. OK for mainline. Thanks, Uros.
