[Mlir-commits] [llvm] [mlir] mlir/MathExtras: consolidate with llvm/MathExtras (PR #95087)

Nikita Popov llvmlistbot at llvm.org
Tue Jun 11 12:45:52 PDT 2024


================
@@ -114,7 +114,7 @@ inline int64_t shardDimension(int64_t dimSize, int64_t shardCount) {
     return ShapedType::kDynamic;
 
   assert(dimSize % shardCount == 0);
-  return ceilDiv(dimSize, shardCount);
+  return llvm::divideCeilSigned(dimSize, shardCount);
----------------
nikic wrote:

Using divideCeil here doesn't make sense because the preceding assert ensures that the division is exact, so this is equivalent to a normal division.

https://github.com/llvm/llvm-project/pull/95087


More information about the Mlir-commits mailing list