[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-07-17 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

Markus Trippelsdorf trippels at gcc dot gnu.org changed:

   What|Removed |Added

 CC||abensonca at gmail dot com

--- Comment #7 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
*** Bug 61824 has been marked as a duplicate of this bug. ***


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-07-10 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

Markus Trippelsdorf trippels at gcc dot gnu.org changed:

   What|Removed |Added

 CC||slayoo at staszic dot waw.pl

--- Comment #6 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
*** Bug 61774 has been marked as a duplicate of this bug. ***


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-07-07 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Priority|P3  |P1


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-06-26 Thread Ganesh.Gopalasubramanian at amd dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

GGanesh Ganesh.Gopalasubramanian at amd dot com changed:

   What|Removed |Added

 CC||Ganesh.Gopalasubramanian@am
   ||d.com

--- Comment #4 from GGanesh Ganesh.Gopalasubramanian at amd dot com ---
Any update on this?
Almost the entire polyhedron benchmark suite fails with -Ofast -march=bdver3.


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-06-26 Thread zsojka at seznam dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

--- Comment #5 from Zdenek Sojka zsojka at seznam dot cz ---
If anyone is interested in what architecutres are affected without looking at
the source code, there are rough statistics of ICEs encountered since it first
appeared:
ICEs count | switch
 21 -march=bdver3
160 -mtune=amdfam10
134 -mtune=barcelona
153 -mtune=bdver1
129 -mtune=bdver2
176 -mtune=bdver3
145 -mtune=bdver4

The reason for so low -march= count is that I am no longer using the -march=
switch. Other architectures are likely not affected.


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-06-14 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

rsandifo at gcc dot gnu.org rsandifo at gcc dot gnu.org changed:

   What|Removed |Added

 CC||uros at gcc dot gnu.org

--- Comment #3 from rsandifo at gcc dot gnu.org rsandifo at gcc dot gnu.org 
---
This is due to the use of the enabled attribute in:

(define_insn *floatSWI48:modeMODEF:mode2_sse
  [(set (match_operand:MODEF 0 register_operand =f,x,x)
(float:MODEF
  (match_operand:SWI48 1 nonimmediate_operand m,r,m)))]
  SSE_FLOAT_MODE_P (MODEF:MODEmode)  TARGET_SSE_MATH
  @
   fild%Z1\t%1
   %vcvtsi2MODEF:ssemodesuffixSWI48:rex64suffix\t{%1, %d0|%d0, %1}
   %vcvtsi2MODEF:ssemodesuffixSWI48:rex64suffix\t{%1, %d0|%d0, %1}
  [(set_attr type fmov,sseicvt,sseicvt)
   (set_attr prefix orig,maybe_vex,maybe_vex)
   (set_attr mode MODEF:MODE)
   (set (attr prefix_rex)
 (if_then_else
   (and (eq_attr prefix maybe_vex)
(match_test SWI48:MODEmode == DImode))
   (const_string 1)
   (const_string *)))
   (set_attr unit i387,*,*)
   (set_attr athlon_decode *,double,direct)
   (set_attr amdfam10_decode *,vector,double)
   (set_attr bdver1_decode *,double,direct)
   (set_attr fp_int_src true)
   (set (attr enabled)
 (cond [(eq_attr alternative 0)
  (symbol_ref TARGET_MIX_SSE_I387
X87_ENABLE_FLOAT (MODEF:MODEmode,
SWI48:MODEmode))
(eq_attr alternative 1)
  /* ??? For sched1 we need constrain_operands to be able to
 select an alternative.  Leave this enabled before RA.  */
  (symbol_ref TARGET_INTER_UNIT_CONVERSIONS
   || optimize_function_for_size_p (cfun)
   || !(reload_completed
|| reload_in_progress
|| lra_in_progress))
   ]
   (symbol_ref true)))
   ])

enabled was really only supposed to be used to enable or disable
alternatives according to the current subtarget, rather than enable
or disable them based on the current stage in the pass pipeline
or on whether the function is being compiled for size or speed.
I have an idea for handling the size/speed thing, but we need
to fix the ??? as well.


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-06-13 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

Markus Trippelsdorf trippels at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2014-06-13
 CC||trippels at gcc dot gnu.org,
   ||vmakarov at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
Confirmed. Also happens when building Firefox with -march=amdfam10.
Vladimir?


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-06-13 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

Markus Trippelsdorf trippels at gcc dot gnu.org changed:

   What|Removed |Added

 CC||rsandifo at gcc dot gnu.org

--- Comment #2 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
Started with r210964 (doesn't compile) or r210965.


[Bug target/61360] [4.10 Regression] ICE: in lra_update_insn_recog_data, at lra.c:1363 with -mtune=bdver4

2014-06-02 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Target Milestone|--- |4.10.0