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

[Kazu Hirata]

Author: Kazu Hirata
Date: 2022-06-25T09:18:04-07:00
New Revision: f8c1c9afd3e2286a8fac99fb9978f1566b89fa70

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

LOG: [mlir] Fix a warning

This patch fixes:

  llvm-project/mlir/lib/Dialect/Linalg/Transforms/SplitReduction.cpp:300:26:
  error: comparison of integers of different signs: 'int64_t' (aka
  'long') and 'size_t' (aka 'unsigned long') [-Werror,-Wsign-compare]

Added: 
    

Modified: 
    mlir/lib/Dialect/Linalg/Transforms/SplitReduction.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/Linalg/Transforms/SplitReduction.cpp b/mlir/lib/Dialect/Linalg/Transforms/SplitReduction.cpp
index 8834000edd69b..61989f1eb7b29 100644
--- a/mlir/lib/Dialect/Linalg/Transforms/SplitReduction.cpp
+++ b/mlir/lib/Dialect/Linalg/Transforms/SplitReduction.cpp
@@ -297,7 +297,7 @@ FailureOr<SplitReductionResult> mlir::linalg::splitReductionByScaling(
     return b.notifyMatchFailure(op, "unknown reduction neutral");
 
   // TODO: relax this when multi-reduction support is available.
-  if (op.getNumOutputs() != neutralElements.size())
+  if (op.getNumOutputs() != (int)neutralElements.size())
     return b.notifyMatchFailure(op, "expect one reduction per output");
 
   // Rewrite part.