Re: PATCH: PR target/59588: Don't check/change generic/i686 tuning

2013-12-26 Thread H.J. Lu
On Thu, Dec 26, 2013 at 11:11 AM, H.J. Lu wrote: > On Thu, Dec 26, 2013 at 8:06 AM, H.J. Lu wrote: >> On Thu, Dec 26, 2013 at 7:45 AM, Jan Hubicka wrote: On Thu, Dec 26, 2013 at 4:38 AM, Jan Hubicka wrote: >> Hi Honza, >> >> We have combined generic32 and generic64 into gene

Re: PATCH: PR target/59588: Don't check/change generic/i686 tuning

2013-12-26 Thread H.J. Lu
On Thu, Dec 26, 2013 at 8:06 AM, H.J. Lu wrote: > On Thu, Dec 26, 2013 at 7:45 AM, Jan Hubicka wrote: >>> On Thu, Dec 26, 2013 at 4:38 AM, Jan Hubicka wrote: >>> >> Hi Honza, >>> >> >>> >> We have combined generic32 and generic64 into generic. There is no need >>> >> to check "generic" anymore.

Re: PATCH: PR target/59588: Don't check/change generic/i686 tuning

2013-12-26 Thread H.J. Lu
On Thu, Dec 26, 2013 at 7:45 AM, Jan Hubicka wrote: >> On Thu, Dec 26, 2013 at 4:38 AM, Jan Hubicka wrote: >> >> Hi Honza, >> >> >> >> We have combined generic32 and generic64 into generic. There is no need >> >> to check "generic" anymore. Also we shouldn't change -mtune=i686 into >> >> -mtune

Re: PATCH: PR target/59588: Don't check/change generic/i686 tuning

2013-12-26 Thread Jan Hubicka
> On Thu, Dec 26, 2013 at 4:38 AM, Jan Hubicka wrote: > >> Hi Honza, > >> > >> We have combined generic32 and generic64 into generic. There is no need > >> to check "generic" anymore. Also we shouldn't change -mtune=i686 into > >> -mtune=generic. OK to install? > > > > The i686->generic change

Re: PATCH: PR target/59588: Don't check/change generic/i686 tuning

2013-12-26 Thread H.J. Lu
On Thu, Dec 26, 2013 at 4:38 AM, Jan Hubicka wrote: >> Hi Honza, >> >> We have combined generic32 and generic64 into generic. There is no need >> to check "generic" anymore. Also we shouldn't change -mtune=i686 into >> -mtune=generic. OK to install? > > The i686->generic change was intended to

Re: PATCH: PR target/59588: Don't check/change generic/i686 tuning

2013-12-26 Thread Jan Hubicka
> Hi Honza, > > We have combined generic32 and generic64 into generic. There is no need > to check "generic" anymore. Also we shouldn't change -mtune=i686 into > -mtune=generic. OK to install? The i686->generic change was intended to get generic optimized code for i686-linux configuration rath

PATCH: PR target/59588: Don't check/change generic/i686 tuning

2013-12-24 Thread H.J. Lu
Hi Honza, We have combined generic32 and generic64 into generic. There is no need to check "generic" anymore. Also we shouldn't change -mtune=i686 into -mtune=generic. OK to install? Thanks. H.J. --- gcc/ 2013-12-24 H.J. Lu PR target/59588 * config/i386/i386.c (ix86_opti