[PATCH] D109555: [APInt] Enable APInt to support zero bit integers.

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 6 01:17:46 PDT 2021


foad added a comment.

In D109555#3043950 <https://reviews.llvm.org/D109555#3043950>, @lattner wrote:

> I'm getting caught back up on old patches and I'm not sure if I closed this off with you.  `isAllOnes()` seems defined on zero bit values now, so I think you're happy.  Am I missing anything more?

isAllOnes returns false on zero bit values. I still maintain it should return true. Then I'd be happy :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109555



More information about the llvm-commits mailing list