Re: [PATCH][AArch64] Add support for fused compare and branch

2019-12-04 Thread Richard Sandiford
Wilco Dijkstra writes: > Hi Richard, > >> But what uses CMP_BRANCH after the patch? It looked like you renamed >> all existing uses and didn't add any new ones. > > My next patch will be adding uses of it now I've done some benchmarking > to decide when to turn it on. > >>> + && reg_referenc

Re: [PATCH][AArch64] Add support for fused compare and branch

2019-12-03 Thread Wilco Dijkstra
Hi Richard, > But what uses CMP_BRANCH after the patch?  It looked like you renamed > all existing uses and didn't add any new ones. My next patch will be adding uses of it now I've done some benchmarking to decide when to turn it on. >> +  && reg_referenced_p (SET_DEST (curr_set), PATTERN (

Re: [PATCH][AArch64] Add support for fused compare and branch

2019-11-29 Thread Richard Sandiford
Wilco Dijkstra writes: > Hi, > > Add support for fused compare with branch. Rename the existing > AARCH64_FUSE_CMP_BRANCH to ALU_BRANCH, and AARCH64_FUSE_ALU_BRANCH > to ALU_CBZ to make it clear what is being fused. This night have been easier to review as three patches: (1) rename ALU_BRANCH t