[PATCH] D128911: Emit table lookup from TargetLowering::expandCTTZ()

Shubham Narlawar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 15 04:48:41 PDT 2022


gsocshubham added a comment.

I will send revised patch shortly to resolve below comments -

a. Use of APInt for DeBruijn instead of unsigned int.
b. Use `ROTL` instead of `<<`.
c. Simplify creation of constant data array in constant pool.


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

https://reviews.llvm.org/D128911



More information about the llvm-commits mailing list