[PATCH] D59959: [ConstantRange] Add unsigned intersection type

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 28 15:15:17 PDT 2019


lebedev.ri added inline comments.


================
Comment at: llvm/unittests/IR/ConstantRangeTest.cpp:400
 
         assert(!HaveInterrupt3 && "Should have at most three ranges");
 
----------------
This looks inconsistent?
`"Should have at most *two* ranges"` maybe?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59959





More information about the llvm-commits mailing list