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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 9 08:41:40 PDT 2019


spatel added a comment.

In D63004#1535595 <https://reviews.llvm.org/D63004#1535595>, @xbolva00 wrote:

> @spatel Thanks for review
>
> What about current revision? Is it OK for you?


Seems ok, but you didn't answer the request to add tests for another target. We want to have more confidence that this is universally good, so something besides x86 should be tested.


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

https://reviews.llvm.org/D63004





More information about the llvm-commits mailing list