[PATCH] D63004: [TargetLowering] Simplify (ctpop x) == 1

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 7 13:28:31 PDT 2019


xbolva00 added a comment.

In D63004#1534364 <https://reviews.llvm.org/D63004#1534364>, @RKSimon wrote:

> We need to check other targets if we do this as a generic combine


Since we have already implemented tranformations in TargetLowering like:
// (ctpop x) u< 2 -> (x & x-1) == 0
// (ctpop x) u> 1 -> (x & x-1) != 0

I think I just should take this first version (was suggested in the comment) and go ahead with review...


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

https://reviews.llvm.org/D63004





More information about the llvm-commits mailing list