more tidying of x86 feature tests

Uros Bizjak ubizjak@gmail.com
Wed Mar 7 16:09:00 GMT 2007


On 3/7/07, H. J. Lu <hjl@lucon.org> wrote:

> > >So I guess the conditional above can be enabled iff we are 64bit and
> > >arch is specified as "x86-64" or "nocona"
> >
> > Attached patch implements this proposal.
> >
> > 2007-03-07  Uros Bizjak  <ubizjak@gmail.com>
> >
> >        * config/i386/i386.c (ix86_tune_features) [X86_TUNE_SAHF]:
> >       Also enable for m_K8 and m_AMDFAM10.
> >       (override_options): Do not unconditionally disable
> >       TARGET_USE_SAHF for 64-bit.
> >       * config/i386/i386.md (x86_sahf_1): Do not disable for
> >       TARGET_64BIT.
> >
> > Patch was tested on i686-pc-linux-gnu. If there are no objections,
> > I'll commit this patch to SVN shortly.
> >
>
> Will -mtune=k8 use lahf with this patch?

Uh, yes. I missed one spot in ix86_emit_fp_unordered_jump(). I'll
correct the patch.

BTW: Does CORE2 also prefer sahf?

uros.



More information about the Gcc-patches mailing list