Summary: | vnmul, fnmul patterns incorrect for -frounding-math | ||
---|---|---|---|
Product: | gcc | Reporter: | Joseph S. Myers <jsm28> |
Component: | target | Assignee: | nsz |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ramana |
Priority: | P3 | Keywords: | wrong-code |
Version: | 4.9.0 | ||
Target Milestone: | 4.9.4 | ||
Host: | Target: | arm*-*-* aarch64*-*-* | |
Build: | Known to work: | ||
Known to fail: | Last reconfirmed: | 2015-07-22 00:00:00 |
Description
Joseph S. Myers
2015-07-01 21:44:13 UTC
Author: nsz Date: Mon Jul 6 11:00:03 2015 New Revision: 225450 URL: https://gcc.gnu.org/viewcvs?rev=225450&root=gcc&view=rev Log: [AArch64] PR target/66731 Fix fnmul insn with -frounding-math gcc/Changelog: 2015-07-03 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * config/aarch64/aarch64.md (fnmul<mode>3): Handle -frounding-math. gcc/testsuite/Changelog: 2015-07-03 Szabolcs Nagy <szabolcs.nagy@arm.com> * gcc.target/aarch64/fnmul-1.c: New. * gcc.target/aarch64/fnmul-2.c: New. * gcc.target/aarch64/fnmul-3.c: New. * gcc.target/aarch64/fnmul-4.c: New. Added: trunk/gcc/testsuite/gcc.target/aarch64/fnmul-1.c trunk/gcc/testsuite/gcc.target/aarch64/fnmul-2.c trunk/gcc/testsuite/gcc.target/aarch64/fnmul-3.c trunk/gcc/testsuite/gcc.target/aarch64/fnmul-4.c Modified: trunk/gcc/ChangeLog trunk/gcc/config/aarch64/aarch64.md trunk/gcc/testsuite/ChangeLog Author: nsz Date: Wed Jul 15 09:03:15 2015 New Revision: 225810 URL: https://gcc.gnu.org/viewcvs?rev=225810&root=gcc&view=rev Log: Add missing PR target/66731 to gcc/testsuite/Changelog Modified: trunk/gcc/testsuite/ChangeLog Confirmed. Author: nsz Date: Mon Aug 3 11:12:00 2015 New Revision: 226496 URL: https://gcc.gnu.org/viewcvs?rev=226496&root=gcc&view=rev Log: [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. Added: trunk/gcc/testsuite/gcc.target/arm/vnmul-1.c trunk/gcc/testsuite/gcc.target/arm/vnmul-2.c trunk/gcc/testsuite/gcc.target/arm/vnmul-3.c trunk/gcc/testsuite/gcc.target/arm/vnmul-4.c Modified: trunk/gcc/ChangeLog trunk/gcc/config/arm/arm.c trunk/gcc/config/arm/vfp.md trunk/gcc/testsuite/ChangeLog Author: nsz Date: Mon Aug 3 14:27:43 2015 New Revision: 226507 URL: https://gcc.gnu.org/viewcvs?rev=226507&root=gcc&view=rev Log: Backport form mainline r226496. gcc: Backport form mainline r226496. 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. gcc/testsuite: Backport form mainline r226496. 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. Added: branches/gcc-5-branch/gcc/testsuite/gcc.target/arm/vnmul-1.c branches/gcc-5-branch/gcc/testsuite/gcc.target/arm/vnmul-2.c branches/gcc-5-branch/gcc/testsuite/gcc.target/arm/vnmul-3.c branches/gcc-5-branch/gcc/testsuite/gcc.target/arm/vnmul-4.c Modified: branches/gcc-5-branch/gcc/ChangeLog branches/gcc-5-branch/gcc/config/arm/arm.c branches/gcc-5-branch/gcc/config/arm/vfp.md branches/gcc-5-branch/gcc/testsuite/ChangeLog Author: nsz Date: Mon Aug 3 17:04:29 2015 New Revision: 226519 URL: https://gcc.gnu.org/viewcvs?rev=226519&root=gcc&view=rev Log: Backport form mainline r226496. gcc: Backport form mainline r226496. 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. gcc/testsuite: Backport form mainline r226496. 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. Added: branches/gcc-4_9-branch/gcc/testsuite/gcc.target/arm/vnmul-1.c branches/gcc-4_9-branch/gcc/testsuite/gcc.target/arm/vnmul-2.c branches/gcc-4_9-branch/gcc/testsuite/gcc.target/arm/vnmul-3.c branches/gcc-4_9-branch/gcc/testsuite/gcc.target/arm/vnmul-4.c Modified: branches/gcc-4_9-branch/gcc/ChangeLog branches/gcc-4_9-branch/gcc/config/arm/arm.c branches/gcc-4_9-branch/gcc/config/arm/vfp.md branches/gcc-4_9-branch/gcc/testsuite/ChangeLog Author: nsz Date: Tue Aug 4 16:22:32 2015 New Revision: 226586 URL: https://gcc.gnu.org/viewcvs?rev=226586&root=gcc&view=rev Log: [AArch64] PR target/66731 Fix fnmul insn with -frounding-math (rtx costs) 2015-08-04 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * config/aarch64/aarch64.c (aarch64_rtx_costs): Fix NEG cost for FNMUL. (aarch64_rtx_mult_cost): Fix MULT cost with -frounding-math. Modified: trunk/gcc/ChangeLog trunk/gcc/config/aarch64/aarch64.c Author: nsz Date: Tue Aug 4 16:43:46 2015 New Revision: 226587 URL: https://gcc.gnu.org/viewcvs?rev=226587&root=gcc&view=rev Log: gcc: Backport from mainline: 2015-08-04 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * config/aarch64/aarch64.c (aarch64_rtx_costs): Fix NEG cost for FNMUL. (aarch64_rtx_mult_cost): Fix MULT cost with -frounding-math. 2015-07-06 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * config/aarch64/aarch64.md (fnmul<mode>3): Handle -frounding-math. gcc/testsuite: Backport from mainline r225450: 2015-07-06 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * gcc.target/aarch64/fnmul-1.c: New. * gcc.target/aarch64/fnmul-2.c: New. * gcc.target/aarch64/fnmul-3.c: New. * gcc.target/aarch64/fnmul-4.c: New. Added: branches/gcc-5-branch/gcc/testsuite/gcc.target/aarch64/fnmul-1.c branches/gcc-5-branch/gcc/testsuite/gcc.target/aarch64/fnmul-2.c branches/gcc-5-branch/gcc/testsuite/gcc.target/aarch64/fnmul-3.c branches/gcc-5-branch/gcc/testsuite/gcc.target/aarch64/fnmul-4.c Modified: branches/gcc-5-branch/gcc/ChangeLog branches/gcc-5-branch/gcc/config/aarch64/aarch64.c branches/gcc-5-branch/gcc/config/aarch64/aarch64.md branches/gcc-5-branch/gcc/testsuite/ChangeLog Author: nsz Date: Tue Aug 4 16:49:54 2015 New Revision: 226588 URL: https://gcc.gnu.org/viewcvs?rev=226588&root=gcc&view=rev Log: Fix broken backport patch. gcc: Backport from mainline: 2015-08-04 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * config/aarch64/aarch64.c (aarch64_rtx_costs): Fix NEG cost for FNMUL. (aarch64_rtx_mult_cost): Fix MULT cost with -frounding-math. Modified: branches/gcc-5-branch/gcc/config/aarch64/aarch64.c Author: nsz Date: Tue Aug 4 17:42:05 2015 New Revision: 226592 URL: https://gcc.gnu.org/viewcvs?rev=226592&root=gcc&view=rev Log: gcc: Backport from mainline: 2015-07-06 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * config/aarch64/aarch64.md (fnmul<mode>3): Handle -frounding-math. gcc/testsuite: Backport from mainline r225450: 2015-07-06 Szabolcs Nagy <szabolcs.nagy@arm.com> PR target/66731 * gcc.target/aarch64/fnmul-1.c: New. * gcc.target/aarch64/fnmul-2.c: New. * gcc.target/aarch64/fnmul-3.c: New. * gcc.target/aarch64/fnmul-4.c: New. Added: branches/gcc-4_9-branch/gcc/testsuite/gcc.target/aarch64/fnmul-1.c branches/gcc-4_9-branch/gcc/testsuite/gcc.target/aarch64/fnmul-2.c branches/gcc-4_9-branch/gcc/testsuite/gcc.target/aarch64/fnmul-3.c branches/gcc-4_9-branch/gcc/testsuite/gcc.target/aarch64/fnmul-4.c Modified: branches/gcc-4_9-branch/gcc/ChangeLog branches/gcc-4_9-branch/gcc/config/aarch64/aarch64.md branches/gcc-4_9-branch/gcc/testsuite/ChangeLog both arm and aarch64 should be fixed now (fix is backported to 4.9 and 5 branches) Fixed for 4.9.4 |