[Bug rtl-optimization/49014] [4.7 Regression] ICE: in reset_sched_cycles_in_current_ebb, at sel-sched.c:7132 with even more insane set of flags

[abel at gcc dot gnu.org]

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

Andrey Belevantsev <abel at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ubizjak at gmail dot com

--- Comment #3 from Andrey Belevantsev <abel at gcc dot gnu.org> 2011-05-19 16:43:04 UTC ---
We again hit an insn with the nothing reservation.  This time it is the compare
from *cmpfp_iu_sse.  It does set the decode attribute to vector correctly but
sets type to ssecomi, and in the athlon.md the athlon_ssecomi reservation
compares against ssecmp, not ssecomi.  I see the following comment there:

;; It seems athlon_ssecomi has a bug in the attr_type, fixed for amdfam10

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")