Re: [PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53

2017-06-06 Thread Ramana Radhakrishnan
On Tue, Jun 6, 2017 at 1:56 PM, James Greenhalgh wrote: > On Tue, May 02, 2017 at 04:37:21PM +0100, Tamar Christina wrote: >> Hi All, >> >> This patch adjusts the cost model for Cortex-A53 to increase the costs of >> an integer division. The reason for this is that we want to always expand >> the

Re: [PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53

2017-06-06 Thread James Greenhalgh
On Tue, May 02, 2017 at 04:37:21PM +0100, Tamar Christina wrote: > Hi All, > > This patch adjusts the cost model for Cortex-A53 to increase the costs of > an integer division. The reason for this is that we want to always expand > the division to a multiply when doing a division by constant. > >

Re: [PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53

2017-05-15 Thread Tamar Christina
; James Greenhalgh; ni...@redhat.com; Ramana Radhakrishnan Subject: Re: [PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53 On Tue, May 2, 2017 at 4:37 PM, Tamar Christina wrote: > Hi All, > > This patch adjusts the cost model for Cortex-A53 to increase the costs of > an inte

Re: [PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53

2017-05-15 Thread Ramana Radhakrishnan
On Tue, May 2, 2017 at 4:37 PM, Tamar Christina wrote: > Hi All, > > This patch adjusts the cost model for Cortex-A53 to increase the costs of > an integer division. The reason for this is that we want to always expand > the division to a multiply when doing a division by constant. > > On the Cort

Re: [PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53

2017-05-15 Thread Tamar Christina
: [PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53 Hi All, This patch adjusts the cost model for Cortex-A53 to increase the costs of an integer division. The reason for this is that we want to always expand the division to a multiply when doing a division by constant. On the Cortex-A53

[PATCH][GCC][AArch64][ARM] Modify idiv costs for Cortex-A53

2017-05-02 Thread Tamar Christina
Hi All, This patch adjusts the cost model for Cortex-A53 to increase the costs of an integer division. The reason for this is that we want to always expand the division to a multiply when doing a division by constant. On the Cortex-A53 shifts are modeled to cost 1 instruction, when doing the exp