[PATCH] D89976: [ValueTracking] add range limits for ctpop

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 22 13:36:14 PDT 2020


spatel added a comment.

In D89976#2348230 <https://reviews.llvm.org/D89976#2348230>, @davezarzycki wrote:

> Thanks. Unless less I missed it, we should also add the upper range for count leading/trailing zero too.

Yes - it should just be adding the `case` line on top of this and adding tests. But I prefer to do it in steps, so it's easier to debug if we hit any problems.


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

https://reviews.llvm.org/D89976



More information about the llvm-commits mailing list