RE: [GCC][PATCH][ARM] Require arm_arch_v8a_ok for sdiv_costs_1.c

2017-06-15 Thread Tamar Christina
> >> * gcc.target/arm/sdiv_costs_1.c: > >> Require arm_arch_v8a_ok and add march option. > >> > > Shouldn't you use > add_options_for_arm_arch_v8a instead? Done, committed as r249214. I don't quite understand the difference, but this test suite does quite counter intuitive things

Re: [GCC][PATCH][ARM] Require arm_arch_v8a_ok for sdiv_costs_1.c

2017-06-13 Thread Christophe Lyon
On 13 June 2017 at 12:13, Kyrill Tkachov wrote: > > On 13/06/17 11:12, Tamar Christina wrote: >> >> Hi All, >> >> This fixes the failing test gcc.target/arm/sdiv_costs_1.c by >> requiring arm_arch_v8a_ok. >> >> >> OK for trunk? >> > > Ok. > Thanks, > Kyrill > > >> gcc/testsuite/ >> 2017-06-13 Tam

Re: [GCC][PATCH][ARM] Require arm_arch_v8a_ok for sdiv_costs_1.c

2017-06-13 Thread Kyrill Tkachov
On 13/06/17 11:12, Tamar Christina wrote: Hi All, This fixes the failing test gcc.target/arm/sdiv_costs_1.c by requiring arm_arch_v8a_ok. OK for trunk? Ok. Thanks, Kyrill gcc/testsuite/ 2017-06-13 Tamar Christina * gcc.target/arm/sdiv_costs_1.c: Require arm_arch_v8a_o

[GCC][PATCH][ARM] Require arm_arch_v8a_ok for sdiv_costs_1.c

2017-06-13 Thread Tamar Christina
Hi All, This fixes the failing test gcc.target/arm/sdiv_costs_1.c by requiring arm_arch_v8a_ok. OK for trunk? gcc/testsuite/ 2017-06-13 Tamar Christina * gcc.target/arm/sdiv_costs_1.c: Require arm_arch_v8a_ok and add march option. Thanks, Tamardiff --git a/gcc/testsuite/gcc