[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #11 from Christophe Lyon --- I've added the -mbranch-cost=N option to the arm target, but compiling /gcc.dg/tree-ssa/pr81588.c with -mcpu=cortex-a5 -mfpu=vfpv3-d16 -O2 -fdump-tree-reassoc1-details -mbranch-cost=N -S -o pr81588.s

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #10 from Jakub Jelinek --- The best would be to add -mbranch-cost= support like many other targets have, and have an effective target with the list of targets that do support it, adjust all such testcases to use that.

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #9 from Christophe Lyon --- Sure, I'm just reluctant to add yet another check_effective_target_arm_cortex_a5 function in the already extremely long list of arm-dedicated queries. Maybe I can play with arm's -mprint-tune-info, which

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #8 from Jakub Jelinek --- (In reply to Christophe Lyon from comment #7) > I think it returns 0 in the testcase (optimizing for speed). > > I've noticed that arm_cortex_a5_branch_cost() has the same implementation as >

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #7 from Christophe Lyon --- I think it returns 0 in the testcase (optimizing for speed). I've noticed that arm_cortex_a5_branch_cost() has the same implementation as arm_cortex_m7_branch_cost(), but according to Thomas, the

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #6 from Jakub Jelinek --- Doesn't cortex-a5 also force BRANCH_COST of 1? If yes, you should fix up logical_op_short_circuit tcl function. If not, then I have no idea why that happens.

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 Christophe Lyon changed: What|Removed |Added CC||clyon at gcc dot gnu.org --- Comment

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #4 from Jakub Jelinek --- Author: jakub Date: Wed Sep 6 15:10:28 2017 New Revision: 251806 URL: https://gcc.gnu.org/viewcvs?rev=251806=gcc=rev Log: PR testsuite/82120 * gcc.dg/tree-ssa/pr81588.c: Don't run on

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #3 from Thomas Preud'homme --- (In reply to Jakub Jelinek from comment #2) > This is a total mess. I've copied a boilerplate from other tests that > require branch cost of 2. > I guess > 2017-09-06 Jakub Jelinek

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #2 from Jakub Jelinek --- This is a total mess. I've copied a boilerplate from other tests that require branch cost of 2. I guess 2017-09-06 Jakub Jelinek PR testsuite/82120 *

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 Richard Biener changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment