[Mlir-commits] [mlir] b2674de - [mlir] Fix a warning

[Kazu Hirata]

Author: Kazu Hirata
Date: 2022-10-12T15:25:17-07:00
New Revision: b2674de375033c0c63e65f649bc501a90aa4d74e

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

LOG: [mlir] Fix a warning

This patch fixes:

  mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp:587:27:
  error: comparison of integers of different signs: 'int64_t' (aka
  'long') and 'uint64_t' (aka 'unsigned long')
  [-Werror,-Wsign-compare]

Added: 
    

Modified: 
    mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp b/mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp
index 1d02b37e6704f..a28aa3b72ff43 100644
--- a/mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp
+++ b/mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp
@@ -584,7 +584,7 @@ struct NewRewriter : public OpRewritePattern<NewOp> {
                                  {reader, indices}, EmitCInterface::On)
                       .getResult(0);
     SmallVector<Value, 4> indicesArray;
-    for (int64_t i = 0; i < rank; i++) {
+    for (uint64_t i = 0; i < rank; i++) {
       indicesArray.push_back(rewriter.create<memref::LoadOp>(
           loc, indices, constantIndex(rewriter, loc, i)));
     }