[PATCH] D95959: [ValueTracking] improve analysis for "C << X" and "C >> X"

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 3 10:38:07 PST 2021


spatel added a comment.

In D95959#2539683 <https://reviews.llvm.org/D95959#2539683>, @RKSimon wrote:

> This is very similar to D90479 <https://reviews.llvm.org/D90479> which is trying to do this more generally by using the KnownBits shift helpers.

Ah, I forgot about that patch. Was anything holding it up?
Yes, they're showing almost the same test diffs apart from the zext+icmp test. I don't have a preference for the fix, so we should push one of these.


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

https://reviews.llvm.org/D95959



More information about the llvm-commits mailing list