Re: [PATCH][AArch64] Handle AND+ASHIFT form of UBFIZ correctly in costs

2016-06-14 Thread James Greenhalgh
On Thu, Jun 09, 2016 at 10:36:36AM +0100, Kyrill Tkachov wrote: > Hi all, > > We currently don't handle in the aarch64 rtx costs the pattern > *andim_ashift_bfiz that performs an ASHIFT followed by an AND. So we > end up recursing inside the AND and assigning a high cost to the pattern. Not > high

[PATCH][AArch64] Handle AND+ASHIFT form of UBFIZ correctly in costs

2016-06-09 Thread Kyrill Tkachov
Hi all, We currently don't handle in the aarch64 rtx costs the pattern *andim_ashift_bfiz that performs an ASHIFT followed by an AND. So we end up recursing inside the AND and assigning a high cost to the pattern. Not high enough to reject it during combine, but still wrong. This patch fixes