[PATCH] D62822: [LVI][CVP] Add support for urem, srem and sdiv

[Philip Reames via Phabricator via llvm-commits]

reames requested changes to this revision.
reames added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/IR/ConstantRange.cpp:792
     return multiply(Other);
   default:
+    llvm_unreachable("Unsupported binary op");
----------------
If I understand the context of the patch, xor is the only case not handled right?  If so, then I'd strongly prefer to see a case handled for xor (even if that's simply return full).  The contract of "supports all binops but xor" is not a great API for this function.  


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62822