[PATCH] D119010: [AggressiveInstCombine] Recognize table-based ctz implementation and enable it for AARCH64 at -O3

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 4 10:15:12 PST 2022


dmgreen added a comment.

The changes in AArch64InstrInfo.td need to be their own patch in a separate commit (with their own tests). It's hard to see how they are correct as-is, but if you put up a new patch for them we can sort out what we need to do there.

Patches should also be uploaded with context, as per the llvm docs: https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface


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

https://reviews.llvm.org/D119010



More information about the llvm-commits mailing list