[PATCH] D32637: [KnownBits] Add methods to set and clear all bits.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 4 14:06:17 PDT 2017


RKSimon added a comment.

I'm not sure we have a need for both the 'set' and 'force' versions - AFAICT all the 'set' cases above could be 'force' as we're just making use of the fact that we know that the KnownBits are all zero at initialization. Maybe just have 'force' version and call them 'set'? If that make sense at all......


https://reviews.llvm.org/D32637





More information about the llvm-commits mailing list