[PATCH] D113291: [WIP] Add LowerTableBasedCTZ and enable it for AARCH64 in -O3

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 10 12:12:27 PST 2021


djtodoro added inline comments.


================
Comment at: llvm/include/llvm/InitializePasses.h:282
 void initializeLowerMatrixIntrinsicsMinimalLegacyPassPass(PassRegistry &);
+void initializeLowerTableBasedCTZLegacyPassPass(PassRegistry &);
 void initializeMIRAddFSDiscriminatorsPass(PassRegistry &);
----------------
djtodoro wrote:
> craig.topper wrote:
> > Can you use CTTZ instead of CTZ to match the llvm intrinsic name.
> Sute, typo.
> 
> I am working on moving this to the AgressiveInstCombine.
> 
Sure*


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113291



More information about the llvm-commits mailing list