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

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D113291#3602808 <https://reviews.llvm.org/D113291#3602808>, @efriedma wrote:

> I don't think I have much further to say.  Emitting a table lookup from TargetLowering::expandCTTZ should be straightforward, I think.  See DAGCombiner::convertSelectOfFPConstantsToLoadOffset for an example of how to emit a constant table.

Do we need to be careful with vector CTTZ which can also go through expand CTTZ?


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

https://reviews.llvm.org/D113291