On Mon, Jan 11, 2016 at 04:41:22PM +0000, Kyrill Tkachov wrote: > Hi all, > > The test gcc.target/aarch64/tst_3.c fails for an explicit -mcpu=cortex-a53 > because we don't handle the recent compare with zero_extract pattern properly > in rtx costs, so we end up recursing into its operands and end up rejecting > the combination for some CPUs, generating an AND-immediate followed by a > comparison against zero, instead of the TST-immediate instruction expected by > the test. > > This patch adds handling for that pattern so that we properly handle it the > same ways as an ANDS instruction. With this patch the aforementioned test > passes for -mcpu=cortex-a53 as well. > > Bootstrapped and tested on aarch64-none-linux-gnu. > Ok for trunk?
OK. Thanks, James > > Thanks, > Kyrill > > 2016-01-11 Kyrylo Tkachov <kyrylo.tkac...@arm.com> > > * config/aarch64/aarch64.c (aarch64_rtx_costs, COMPARE case): > Handle COMPARE of ZERO_EXTRACT against zero form of TST-immediate.