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

[Djordje Todorovic via Phabricator via llvm-commits]

djtodoro added a comment.

In D113291#3569114 <https://reviews.llvm.org/D113291#3569114>, @gsocshubham wrote:

> 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."?

Unfortunately, I don’t have time to do it right now. If you are interested, please go ahead with the implementation.


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

https://reviews.llvm.org/D113291