[PATCH] D21610: [ValueTracking] improve ComputeNumSignBits for vectors

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 22 10:35:33 PDT 2016


eli.friedman accepted this revision.
eli.friedman added a comment.
This revision is now accepted and ready to land.

LGTM.

I did a bit of digging about the "Mask <<= Mask.getBitWidth()-TyBits;" thing; it looks like r47654 is at fault.  (The original version of the code used uint64_t instead of APInt.)


http://reviews.llvm.org/D21610





More information about the llvm-commits mailing list