http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49014

--- Comment #4 from Uros Bizjak <ubizjak at gmail dot com> 2011-05-19 19:09:44 
UTC ---
(In reply to comment #3)

> Changing the type to ssecomi (like below, as in atom.md) fixes this.
> 
> Uros, does this look right to you?  The other ssecomi* reservations also have
> the ssecomi type.  If yes, then I will add the assert to the scheduler which
> will fire immediately upon detecting this, it will be simple. 
> 
> 
> diff --git a/gcc/config/i386/athlon.md b/gcc/config/i386/athlon.md
> index 341adf4..2896a15 100644
> --- a/gcc/config/i386/athlon.md
> +++ b/gcc/config/i386/athlon.md
> @@ -798,7 +798,7 @@
>                          "athlon-direct,athlon-fploadk8,athlon-fadd")
>  (define_insn_reservation "athlon_ssecomi" 4
>                          (and (eq_attr "cpu" "athlon,k8,generic64")
> -                             (eq_attr "type" "ssecmp"))
> +                             (eq_attr "type" "ssecomi"))
>                          "athlon-vector,athlon-fpsched,athlon-fadd")
>  (define_insn_reservation "athlon_ssecomi_amdfam10" 3
>                          (and (eq_attr "cpu" "amdfam10")

This change looks correct to me, "ssecmp" type goes to ssecmp* reservations.

Reply via email to