[Mlir-commits] [mlir] 7740010 - NfC fix comment in #119039 (#119727)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Dec 12 09:36:16 PST 2024


Author: Nirvedh Meshram
Date: 2024-12-12T11:36:12-06:00
New Revision: 77400103da63118e433cbee1feb20146a9fb782c

URL: https://github.com/llvm/llvm-project/commit/77400103da63118e433cbee1feb20146a9fb782c
DIFF: https://github.com/llvm/llvm-project/commit/77400103da63118e433cbee1feb20146a9fb782c.diff

LOG: NfC fix comment in #119039 (#119727)

Missed commiting clang-fomrat in
[#19903](https://github.com/llvm/llvm-project/pull/119039)

Added: 
    

Modified: 
    mlir/lib/Dialect/Tensor/IR/TensorTilingInterfaceImpl.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/Tensor/IR/TensorTilingInterfaceImpl.cpp b/mlir/lib/Dialect/Tensor/IR/TensorTilingInterfaceImpl.cpp
index 3caf93b1408df4..6e63cf068bce57 100644
--- a/mlir/lib/Dialect/Tensor/IR/TensorTilingInterfaceImpl.cpp
+++ b/mlir/lib/Dialect/Tensor/IR/TensorTilingInterfaceImpl.cpp
@@ -822,7 +822,13 @@ FailureOr<TilingResult> tensor::bubbleUpPadSlice(OpBuilder &b,
     // the source tensor. (Similar to newOffset.)
     // srcSize - newOffset represents how much length we have available
     // and length - newLow represents how much length we want at most.
-    // Note that there are many ways to order this indexing math to compute newLength, but we want to make sure that the final affine.min ops in the sequence are bounding the index to as small a value as possible. If ValueBoundsOpInterface is used, this calcuation will get upper bounds from the affine.min ops, so we want to use the smallest known value to set the bound at the end of the computation sequence. In this case, the index will be upper bounded by length - newLow.
+    // Note that there are many ways to order this indexing math to compute
+    // newLength, but we want to make sure that the final affine.min ops in the
+    // sequence are bounding the index to as small a value as possible. If
+    // ValueBoundsOpInterface is used, this calcuation will get upper bounds
+    // from the affine.min ops, so we want to use the smallest known value to
+    // set the bound at the end of the computation sequence. In this case, the
+    // index will be upper bounded by length - newLow.
     OpFoldResult newLength = min(sub(srcSize, newOffset), sub(length, newLow));
     // Optimization: If low = 0, then newLow = 0. then newLength >= 0 assuming
     // length >= 0.


        


More information about the Mlir-commits mailing list