Re: [PATCH] AArch64: Add TARGET_SCHED_REASSOCIATION_WIDTH

2014-12-09 Thread Marcus Shawcroft
On 24 November 2014 at 13:46, Wilco Dijkstra wdijk...@arm.com wrote: Richard Earnshaw wrote: If all cores seem to benefit from FP reassociation set to 4, then it seems odd that 4 is not also the default for generic. Andrew, you may need to pick a target-specific value for ThunderX; I think

RE: [PATCH] AArch64: Add TARGET_SCHED_REASSOCIATION_WIDTH

2014-12-09 Thread Wilco Dijkstra
Marcus Shawcroft wrote: OK for commit? ChangeLog: 2014-11-24 Wilco Dijkstra wdijk...@arm.com * gcc/config/aarch64/aarch64-protos.h (tune-params): Add reasociation tuning parameters. * gcc/config/aarch64/aarch64.c (TARGET_SCHED_REASSOCIATION_WIDTH):

RE: [PATCH] AArch64: Add TARGET_SCHED_REASSOCIATION_WIDTH

2014-11-24 Thread Wilco Dijkstra
Richard Earnshaw wrote: If all cores seem to benefit from FP reassociation set to 4, then it seems odd that 4 is not also the default for generic. Andrew, you may need to pick a target-specific value for ThunderX; I think Wilco has just picked something that seems plausible because he

Re: [PATCH] AArch64: Add TARGET_SCHED_REASSOCIATION_WIDTH

2014-11-07 Thread Richard Earnshaw
On 29/10/14 12:55, Wilco Dijkstra wrote: This patch adds the TARGET_SCHED_REASSOCIATION_WIDTH hook. Separate settings for integer, floating point and vector modes are supported via the CPU tuning parameters. Setting the FP reassociation width to 4 improves FP performance on SPEC2000 by