[PATCH] D94457: [AArch64] Add some missing fusion subtarget features

Alexander Shaposhnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 12 16:57:45 PDT 2022


alexander-shaposhnikov added subscribers: dmgreen, alexander-shaposhnikov.
alexander-shaposhnikov added a comment.
Herald added a project: All.

@SjoerdMeijer,  @porglezomp - many thanks in advance, 
I'd like to clarify one aspect related to this diff- is it intentional that currently only pairs of cmp + bcc get fused (but e.g. not cmp + cbz or cmp + cbnz) (unless I'm missing something) 
and another question - after looking at https://github.com/gcc-mirror/gcc/blob/master/gcc/config/aarch64/aarch64.cc#L1326 - would it make sense to enable CmpBccFusion by default.

cc: @dmgreen


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94457/new/

https://reviews.llvm.org/D94457



More information about the llvm-commits mailing list