Re: [PATCH][AArch64] Handle CSEL of zero_extended operands in rtx costs

2016-01-15 Thread James Greenhalgh
On Mon, Jan 11, 2016 at 04:41:32PM +, Kyrill Tkachov wrote: > Hi all, > > This patch fixes the test gcc.target/aarch64/pr66776.c for -mcpu=cortex-a53. > Currently we don't handle the (if_then_else (cond) (zero_extend r1) > (zero_extend r2)) > form of CSEL, so we end up recursing into the

[PATCH][AArch64] Handle CSEL of zero_extended operands in rtx costs

2016-01-11 Thread Kyrill Tkachov
Hi all, This patch fixes the test gcc.target/aarch64/pr66776.c for -mcpu=cortex-a53. Currently we don't handle the (if_then_else (cond) (zero_extend r1) (zero_extend r2)) form of CSEL, so we end up recursing into the operands of the if_then_else and for some CPUs reject the combination. We end