[PATCH] D113291: [AggressiveInstCombine] Lower Table Based CTTZ

[Shubham Narlawar via Phabricator via llvm-commits]

gsocshubham added a comment.

In D113291#3566581 <https://reviews.llvm.org/D113291#3566581>, @djtodoro wrote:

> In D113291#3542941 <https://reviews.llvm.org/D113291#3542941>, @efriedma wrote:
>
>> Concretely, my preferred solution looks something like:
>>
>> 1. Perform the transform unconditionally in AggressiveInstCombine (so this patch without the preferCTTZLowering() bits).
>
> The latest update implements this.
>
>> 2. Teach TargetLowering::expandCTTZ to emit a table lookup.

@djtodoro - Will you be sending patch for (2) "Teach TargetLowering::expandCTTZ to emit a table lookup."?


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

https://reviews.llvm.org/D113291