[PATCH] D113291: [AggressiveInstCombine] Lower Table Based CTTZ and enable it for AARCH64 in -O3

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 12 08:11:45 PST 2021


djtodoro updated this revision to Diff 386859.
djtodoro retitled this revision from "[WIP] Add LowerTableBasedCTZ and enable it for AARCH64 in -O3" to "[AggressiveInstCombine] Lower Table Based CTTZ and enable it for AARCH64 in -O3".
djtodoro added a comment.

- Move the functionality into the AggressiveInstCombine (by enabling it for AARCH64 only via TTI)
- Addressing comments
- Fix the 0 element case
- Clean up the code a bit
- Add some additional tests


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

https://reviews.llvm.org/D113291

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/include/llvm/CodeGen/BasicTTIImpl.h
  llvm/include/llvm/CodeGen/TargetLowering.h
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
  llvm/lib/Target/AArch64/AArch64ISelLowering.h
  llvm/lib/Transforms/AggressiveInstCombine/AggressiveInstCombine.cpp
  llvm/test/Transforms/AggressiveInstCombine/AARCH64/dereferencing-pointer.ll
  llvm/test/Transforms/AggressiveInstCombine/AARCH64/lit.local.cfg
  llvm/test/Transforms/AggressiveInstCombine/AARCH64/lower-table-based-ctz-basics.ll
  llvm/test/Transforms/AggressiveInstCombine/AARCH64/lower-table-based-ctz.ll
  llvm/test/Transforms/AggressiveInstCombine/AARCH64/non-argument-value.ll
  llvm/test/Transforms/AggressiveInstCombine/AARCH64/zero-element.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D113291.386859.patch
Type: text/x-patch
Size: 38169 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211112/df7c5dca/attachment.bin>


More information about the llvm-commits mailing list