RE: [PATCH][AArch64] Adjust generic move costs

2014-11-19 Thread Wilco Dijkstra 
Hi Jiong,

Can you commit this please?

2014-11-19  Wilco Dijkstra  wdijk...@arm.com

* gcc/config/aarch64/aarch64.c (generic_regmove_cost):
Increase FP move cost (PR61915).


---
 gcc/config/aarch64/aarch64.c | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index de53c94..cd30724 100644
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -218,8 +218,10 @@ __extension__
 static const struct cpu_regmove_cost generic_regmove_cost =
 {
   NAMED_PARAM (GP2GP, 1),
-  NAMED_PARAM (GP2FP, 2),
-  NAMED_PARAM (FP2GP, 2),
+  /* Avoid the use of slow int-fp moves for spilling by setting
+ their cost higher than memmov_cost.  */
+  NAMED_PARAM (GP2FP, 5),
+  NAMED_PARAM (FP2GP, 5),
   NAMED_PARAM (FP2FP, 2)
 };
 
-- 
1.9.1

Re: [PATCH][AArch64] Adjust generic move costs

2014-11-19 Thread Marcus Shawcroft 
On 19 November 2014 14:32, Wilco Dijkstra wdijk...@arm.com wrote:
 Hi Jiong,

 Can you commit this please?

 2014-11-19  Wilco Dijkstra  wdijk...@arm.com

 * gcc/config/aarch64/aarch64.c (generic_regmove_cost):
 Increase FP move cost (PR61915).


Use the proper format for referring to PR's in the Changelog.

/M

Re: [PATCH][AArch64] Adjust generic move costs

2014-11-18 Thread Ramana Radhakrishnan 
On Mon, Nov 17, 2014 at 5:13 PM, Marcus Shawcroft
marcus.shawcr...@gmail.com wrote:
 On 14 November 2014 14:35, Wilco Dijkstra wdijk...@arm.com wrote:

 2014-11-14  Wilco Dijkstra  wdijk...@arm.com

 * gcc/config/aarch64/aarch64.c (generic_regmove_cost):
 Increase FP move cost.

 OK /Marcus

Changelog should probably indicate PR target/61915 when committing ?

Ramana

Re: [PATCH][AArch64] Adjust generic move costs

2014-11-17 Thread Marcus Shawcroft 
On 14 November 2014 14:35, Wilco Dijkstra wdijk...@arm.com wrote:

 2014-11-14  Wilco Dijkstra  wdijk...@arm.com

 * gcc/config/aarch64/aarch64.c (generic_regmove_cost):
 Increase FP move cost.

OK /Marcus