[PATCH] D60952: [ConstantRange] Add urem support

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 22 11:01:10 PDT 2019


lebedev.ri added a comment.

While i understand that you can't do a traditional exhaustive test that this produces *identical* results,
can't you at least test that the `ConstantRange::urem()` is still **always** //conservatively// correct?


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

https://reviews.llvm.org/D60952





More information about the llvm-commits mailing list