Re: [PATCH][AArch64] Fix operand costing logic for MINUS

2015-05-01 Thread Kyrill Tkachov
On 30/04/15 17:02, Marcus Shawcroft wrote: On 27 April 2015 at 14:24, Kyrill Tkachov wrote: 2015-04-27 Kyrylo Tkachov * config/aarch64/aarch64.c (aarch64_rtx_costs, MINUS): Properly account for both operand costs in simple case. OK /Marcus Thanks, but my patch is just a subse

Re: [PATCH][AArch64] Fix operand costing logic for MINUS

2015-04-30 Thread Marcus Shawcroft
On 27 April 2015 at 14:24, Kyrill Tkachov wrote: > 2015-04-27 Kyrylo Tkachov > > * config/aarch64/aarch64.c (aarch64_rtx_costs, MINUS): > Properly account for both operand costs in simple case. OK /Marcus

[PATCH][AArch64] Fix operand costing logic for MINUS

2015-04-27 Thread Kyrill Tkachov
Hi all, This fixes a case in aarch64 costs where we forgot to account for one of the operands in the MINUS case. This is important to get right as the mult synthesis code can ask for the cost of a shift+sub operation and put the shift part in any of the MINUS operands, expecting the cost functio