[PATCH] D76446: [ConstantRange] Use APInt::or/APInt::and for single elements.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 13:42:09 PDT 2020


nikic added a comment.

I'd like to see a unit test that exhaustively tests this for all binary operators (or the ones where it holds). I don't think single element ranges are handled accurately for all the ops (e.g. probably not for urem/srem?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76446





More information about the llvm-commits mailing list