Re: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Kyrill Tkachov
On 01/05/15 14:11, Wilco Dijkstra wrote: Marcus Shawcroft wrote: On 1 May 2015 at 12:26, Wilco Dijkstra wrote: Marcus Shawcroft wrote: On 27 April 2015 at 14:43, Wilco Dijkstra wrote: static unsigned int -aarch64_min_divisions_for_recip_mul (enum machine_mode mode ATTRIBUTE_UNUSED) +aar

RE: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Wilco Dijkstra
> Marcus Shawcroft wrote: > On 1 May 2015 at 12:26, Wilco Dijkstra wrote: > > > > > >> Marcus Shawcroft wrote: > >> On 27 April 2015 at 14:43, Wilco Dijkstra wrote: > >> > >> >> static unsigned int > >> >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode > >> >> ATTRIBUTE_UNUSED) >

Re: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Marcus Shawcroft
On 1 May 2015 at 12:26, Wilco Dijkstra wrote: > > >> Marcus Shawcroft wrote: >> On 27 April 2015 at 14:43, Wilco Dijkstra wrote: >> >> >> static unsigned int >> >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode >> >> ATTRIBUTE_UNUSED) >> >> +aarch64_min_divisions_for_recip_mul (en

RE: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Wilco Dijkstra
> Marcus Shawcroft wrote: > On 27 April 2015 at 14:43, Wilco Dijkstra wrote: > > >> static unsigned int > >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode > >> ATTRIBUTE_UNUSED) > >> +aarch64_min_divisions_for_recip_mul (enum machine_mode mode) > >> { > >> - return 2; > >> +

Re: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Marcus Shawcroft
On 27 April 2015 at 14:43, Wilco Dijkstra wrote: >> static unsigned int >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode >> ATTRIBUTE_UNUSED) >> +aarch64_min_divisions_for_recip_mul (enum machine_mode mode) >> { >> - return 2; >> + if (GET_MODE_UNIT_SIZE (mode) == 4) >> +r

RE: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-04-27 Thread Wilco Dijkstra
ping > -Original Message- > From: Wilco Dijkstra [mailto:wdijk...@arm.com] > Sent: 03 March 2015 18:06 > To: GCC Patches > Subject: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul > configurable > > This patch makes aarch64_min_divisions_for_recip_mu

RE: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-03-03 Thread Wilco Dijkstra
> Andrew Pinski wrote: > On Tue, Mar 3, 2015 at 10:06 AM, Wilco Dijkstra wrote: > > This patch makes aarch64_min_divisions_for_recip_mul configurable for float > > and double. This > allows > > CPUs with really fast or multiple dividers to return 3 (or even 4) if that > > happens to be > faster

Re: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-03-03 Thread Andrew Pinski
On Tue, Mar 3, 2015 at 10:06 AM, Wilco Dijkstra wrote: > This patch makes aarch64_min_divisions_for_recip_mul configurable for float > and double. This allows > CPUs with really fast or multiple dividers to return 3 (or even 4) if that > happens to be faster > overall. No code generation change

[PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-03-03 Thread Wilco Dijkstra
This patch makes aarch64_min_divisions_for_recip_mul configurable for float and double. This allows CPUs with really fast or multiple dividers to return 3 (or even 4) if that happens to be faster overall. No code generation change - bootstrap & regression OK. ChangeLog: 2015-03-03 Wilco Dijkst