re PR target/70086 (ICE: in extract_constrain_insn_cached, at recog.c:2202 (insn…

re PR target/70086 (ICE: in extract_constrain_insn_cached, at recog.c:2202 (insn does not satisfy its constraints) with -mavx512vl -ffloat-store)

	PR target/70086
	* config/i386/i386.md (truncdfsf2 splitter): Use gen_vec_concatv2df
	instead of gen_sse2_loadlpd.
	* config/i386/sse.md (*vec_concatv2df): Rename to...
	(vec_concatv2df): ... this.

	* gcc.target/i386/pr70086-1.c: New test.
	* gcc.target/i386/pr70086-2.c: New test.
	* gcc.target/i386/pr70086-3.c: New test.

From-SVN: r234091

gcc/ChangeLog

-2016-03-08  Jakub Jelinek  <jakub@redhat.com>
+2016-03-09  Jakub Jelinek  <jakub@redhat.com>
+
+	PR target/70086
+	* config/i386/i386.md (truncdfsf2 splitter): Use gen_vec_concatv2df
+	instead of gen_sse2_loadlpd.
+	* config/i386/sse.md (*vec_concatv2df): Rename to...
+	(vec_concatv2df): ... this.
 
 	PR tree-optimization/70127
 	* fold-const.c (operand_equal_p): Revert the 2015-10-28 change.

gcc/config/i386/i386.md

@@ -4393,8 +4393,8 @@
       emit_insn (gen_vec_dupv2df (operands[4], operands[1]));
     }
   else
-    emit_insn (gen_sse2_loadlpd (operands[4],
-				 CONST0_RTX (V2DFmode), operands[1]));
+    emit_insn (gen_vec_concatv2df (operands[4], operands[1],
+				   CONST0_RTX (DFmode)));
 })
 
 ;; It's more profitable to split and then extend in the same register.

gcc/config/i386/sse.md

@@ -8951,7 +8951,7 @@
    (set_attr "prefix" "orig,maybe_vex,evex")
    (set_attr "mode" "V2DF,DF,DF")])
 
-(define_insn "*vec_concatv2df"
+(define_insn "vec_concatv2df"
   [(set (match_operand:V2DF 0 "register_operand"     "=x,x,v,x,v,x,x,v,x,x")
 	(vec_concat:V2DF
 	  (match_operand:DF 1 "nonimmediate_operand" " 0,x,v,m,m,0,x,m,0,0")

gcc/testsuite/ChangeLog

-2016-03-08  Jakub Jelinek  <jakub@redhat.com>
+2016-03-09  Jakub Jelinek  <jakub@redhat.com>
+
+	PR target/70086
+	* gcc.target/i386/pr70086-1.c: New test.
+	* gcc.target/i386/pr70086-2.c: New test.
+	* gcc.target/i386/pr70086-3.c: New test.
 
 	PR tree-optimization/70127
 	* gcc.c-torture/execute/pr70127.c: New test.

gcc/testsuite/gcc.target/i386/pr70086-1.c

+/* PR target/70086 */
+/* { dg-do compile } */
+/* { dg-options "-mtune=barcelona -mavx512vl -ffloat-store" } */
+
+float
+foo (float a, float b, double c, float d, double e, float f)
+{
+  e -= d;
+  d *= e;
+  return e + d;
+}

gcc/testsuite/gcc.target/i386/pr70086-2.c

+/* PR target/70086 */
+/* { dg-do compile { target { ! ia32 } } } */
+/* { dg-options "-O2 -mtune=barcelona -mavx512vl" } */
+
+float
+foo (double *p)
+{
+  register float xmm16 __asm ("xmm16");
+  xmm16 = *p;
+  asm volatile ("" : "+v" (xmm16));
+  return xmm16;
+}
+
+float
+bar (double x)
+{
+  register float xmm16 __asm ("xmm16");
+  xmm16 = x;
+  asm volatile ("" : "+v" (xmm16));
+  return xmm16;
+}

gcc/testsuite/gcc.target/i386/pr70086-3.c

+/* PR target/70086 */
+/* { dg-do compile { target { ! ia32 } } } */
+/* { dg-options "-O2 -mtune=barcelona -mavx512f -mno-avx512vl" } */
+
+float
+foo (double *p)
+{
+  register float xmm16 __asm ("xmm16");
+  xmm16 = *p;
+  asm volatile ("" : "+v" (xmm16));
+  return xmm16;
+}
+
+float
+bar (double x)
+{
+  register float xmm16 __asm ("xmm16");
+  xmm16 = x;
+  asm volatile ("" : "+v" (xmm16));
+  return xmm16;
+}