Commit 1f71aee9 by Szabolcs Nagy Committed by Szabolcs Nagy

[ARM] PR target/66731 Fix vnmul insn with -frounding-math

gcc:

	PR target/66731
	* config/arm/vfp.md (negmuldf3_vfp): Add new pattern.
	(negmulsf3_vfp): Likewise.
	(muldf3negdf_vfp): Disable for -frounding-math.
	(mulsf3negsf_vfp): Likewise.
	* config/arm/arm.c (arm_new_rtx_costs): Fix NEG cost for VNMUL,
	fix MULT cost with -frounding-math.

gcc/testsuite:

	PR target/66731
	* gcc.target/arm/vnmul-1.c: New.
	* gcc.target/arm/vnmul-2.c: New.
	* gcc.target/arm/vnmul-3.c: New.
	* gcc.target/arm/vnmul-4.c: New.

From-SVN: r226496
parent 2ab320ad
2015-08-03 Szabolcs Nagy <szabolcs.nagy@arm.com>
PR target/66731
* config/arm/vfp.md (negmuldf3_vfp): Add new pattern.
(negmulsf3_vfp): Likewise.
(muldf3negdf_vfp): Disable for -frounding-math.
(mulsf3negsf_vfp): Likewise.
* config/arm/arm.c (arm_new_rtx_costs): Fix NEG cost for VNMUL,
fix MULT cost with -frounding-math.
2015-08-03 Kyrylo Tkachov <kyrylo.tkachov@arm.com> 2015-08-03 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
* ifcvt.c (noce_try_store_flag_constants): Make logic of the case * ifcvt.c (noce_try_store_flag_constants): Make logic of the case
......
...@@ -10181,7 +10181,7 @@ arm_new_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer_code, ...@@ -10181,7 +10181,7 @@ arm_new_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer_code,
{ {
rtx op0 = XEXP (x, 0); rtx op0 = XEXP (x, 0);
if (GET_CODE (op0) == NEG) if (GET_CODE (op0) == NEG && !flag_rounding_math)
op0 = XEXP (op0, 0); op0 = XEXP (op0, 0);
if (speed_p) if (speed_p)
...@@ -10255,6 +10255,13 @@ arm_new_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer_code, ...@@ -10255,6 +10255,13 @@ arm_new_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer_code,
if (TARGET_HARD_FLOAT && GET_MODE_CLASS (mode) == MODE_FLOAT if (TARGET_HARD_FLOAT && GET_MODE_CLASS (mode) == MODE_FLOAT
&& (mode == SFmode || !TARGET_VFP_SINGLE)) && (mode == SFmode || !TARGET_VFP_SINGLE))
{ {
if (GET_CODE (XEXP (x, 0)) == MULT)
{
/* VNMUL. */
*cost = rtx_cost (XEXP (x, 0), mode, NEG, 0, speed_p);
return true;
}
if (speed_p) if (speed_p)
*cost += extra_cost->fp[mode != SFmode].neg; *cost += extra_cost->fp[mode != SFmode].neg;
......
...@@ -770,6 +770,17 @@ ...@@ -770,6 +770,17 @@
[(set (match_operand:SF 0 "s_register_operand" "=t") [(set (match_operand:SF 0 "s_register_operand" "=t")
(mult:SF (neg:SF (match_operand:SF 1 "s_register_operand" "t")) (mult:SF (neg:SF (match_operand:SF 1 "s_register_operand" "t"))
(match_operand:SF 2 "s_register_operand" "t")))] (match_operand:SF 2 "s_register_operand" "t")))]
"TARGET_32BIT && TARGET_HARD_FLOAT && TARGET_VFP && !flag_rounding_math"
"vnmul%?.f32\\t%0, %1, %2"
[(set_attr "predicable" "yes")
(set_attr "predicable_short_it" "no")
(set_attr "type" "fmuls")]
)
(define_insn "*negmulsf3_vfp"
[(set (match_operand:SF 0 "s_register_operand" "=t")
(neg:SF (mult:SF (match_operand:SF 1 "s_register_operand" "t")
(match_operand:SF 2 "s_register_operand" "t"))))]
"TARGET_32BIT && TARGET_HARD_FLOAT && TARGET_VFP" "TARGET_32BIT && TARGET_HARD_FLOAT && TARGET_VFP"
"vnmul%?.f32\\t%0, %1, %2" "vnmul%?.f32\\t%0, %1, %2"
[(set_attr "predicable" "yes") [(set_attr "predicable" "yes")
...@@ -781,6 +792,18 @@ ...@@ -781,6 +792,18 @@
[(set (match_operand:DF 0 "s_register_operand" "=w") [(set (match_operand:DF 0 "s_register_operand" "=w")
(mult:DF (neg:DF (match_operand:DF 1 "s_register_operand" "w")) (mult:DF (neg:DF (match_operand:DF 1 "s_register_operand" "w"))
(match_operand:DF 2 "s_register_operand" "w")))] (match_operand:DF 2 "s_register_operand" "w")))]
"TARGET_32BIT && TARGET_HARD_FLOAT && TARGET_VFP_DOUBLE
&& !flag_rounding_math"
"vnmul%?.f64\\t%P0, %P1, %P2"
[(set_attr "predicable" "yes")
(set_attr "predicable_short_it" "no")
(set_attr "type" "fmuld")]
)
(define_insn "*negmuldf3_vfp"
[(set (match_operand:DF 0 "s_register_operand" "=w")
(neg:DF (mult:DF (match_operand:DF 1 "s_register_operand" "w")
(match_operand:DF 2 "s_register_operand" "w"))))]
"TARGET_32BIT && TARGET_HARD_FLOAT && TARGET_VFP_DOUBLE" "TARGET_32BIT && TARGET_HARD_FLOAT && TARGET_VFP_DOUBLE"
"vnmul%?.f64\\t%P0, %P1, %P2" "vnmul%?.f64\\t%P0, %P1, %P2"
[(set_attr "predicable" "yes") [(set_attr "predicable" "yes")
......
2015-08-03 Szabolcs Nagy <szabolcs.nagy@arm.com>
PR target/66731
* gcc.target/arm/vnmul-1.c: New.
* gcc.target/arm/vnmul-2.c: New.
* gcc.target/arm/vnmul-3.c: New.
* gcc.target/arm/vnmul-4.c: New.
2015-08-03 Mikael Morin <mikael@gcc.gnu.org> 2015-08-03 Mikael Morin <mikael@gcc.gnu.org>
PR fortran/64921 PR fortran/64921
......
/* { dg-do compile } */
/* { dg-require-effective-target arm_vfp_ok } */
/* { dg-skip-if "need fp instructions" { *-*-* } { "-mfloat-abi=soft" } { "" } } */
/* { dg-options "-O2 -fno-rounding-math -mfpu=vfp -mfloat-abi=hard" } */
double
foo_d (double a, double b)
{
/* { dg-final { scan-assembler "vnmul\\.f64" } } */
return -a * b;
}
float
foo_s (float a, float b)
{
/* { dg-final { scan-assembler "vnmul\\.f32" } } */
return -a * b;
}
/* { dg-do compile } */
/* { dg-require-effective-target arm_vfp_ok } */
/* { dg-skip-if "need fp instructions" { *-*-* } { "-mfloat-abi=soft" } { "" } } */
/* { dg-options "-O2 -frounding-math -mfpu=vfp -mfloat-abi=hard" } */
double
foo_d (double a, double b)
{
/* { dg-final { scan-assembler-not "vnmul\\.f64" } } */
return -a * b;
}
float
foo_s (float a, float b)
{
/* { dg-final { scan-assembler-not "vnmul\\.f32" } } */
return -a * b;
}
/* { dg-do compile } */
/* { dg-require-effective-target arm_vfp_ok } */
/* { dg-skip-if "need fp instructions" { *-*-* } { "-mfloat-abi=soft" } { "" } } */
/* { dg-options "-O2 -fno-rounding-math -mfpu=vfp -mfloat-abi=hard" } */
double
foo_d (double a, double b)
{
/* { dg-final { scan-assembler "vnmul\\.f64" } } */
return -(a * b);
}
float
foo_s (float a, float b)
{
/* { dg-final { scan-assembler "vnmul\\.f32" } } */
return -(a * b);
}
/* { dg-do compile } */
/* { dg-require-effective-target arm_vfp_ok } */
/* { dg-skip-if "need fp instructions" { *-*-* } { "-mfloat-abi=soft" } { "" } } */
/* { dg-options "-O2 -frounding-math -mfpu=vfp -mfloat-abi=hard" } */
double
foo_d (double a, double b)
{
/* { dg-final { scan-assembler "vnmul\\.f64" } } */
return -(a * b);
}
float
foo_s (float a, float b)
{
/* { dg-final { scan-assembler "vnmul\\.f32" } } */
return -(a * b);
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment