[PATCH] D115969: [Support] improve known bits analysis for leading zeros of multiply

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 19 06:48:57 PST 2021


spatel added a comment.

In D115969#3201733 <https://reviews.llvm.org/D115969#3201733>, @lebedev.ri wrote:

> LG
> We *really* need precision tests for KnownBits / ConstantRange.

Thanks!

Side note: I was wondering how we handle the known sign bit for mul; it is not done here, but it is in the caller code in ValueTracking's `computeKnownBitsMul()`. I'm not sure if there's a reason for that.


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

https://reviews.llvm.org/D115969



More information about the llvm-commits mailing list