Reupoading: Apply shortened printing/parsing form to linalg.reduce.

[khasanovaa]

Summary

Original revision: https://reviews.llvm.org/D140622

Revert: https://reviews.llvm.org/rG5759d9467cbec1440be499448bf086db384fd1d2

[khasanovaa]

merge-base

[khasanovaa]

merge-base another try

[olegshyshkov]

mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp

1132

To avoid the index-based loop, you can use a range-based loop over something like llvm::zip(payload.getOperands(), body->getArguments().drop_front()). Or use llvm::all_of_zip.

[khasanovaa]

clang-format

[khasanovaa]

• resolve comemnts

[olegshyshkov]

Thank you!

mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp

static Operation *findPayloadOp(Block *body, bool initFirst = false) {
   if (payload.getNumOperands() == 0 ||
       payload.getNumOperands() != body->getNumArguments())
     return nullptr;
   if (initFirst) {
     // check init
     if (payload.getOperands().back() != body->getArgument(0))
       return nullptr;
     // check rest
-    for (int i = 1; i < body->getNumArguments(); ++i) {
+    for (size_t i = 1; i < body->getNumArguments(); ++i) {
       if (payload.getOperand(i - 1) != body->getArgument(i)) {
         return nullptr;
       }
     }
   } else {
     for (const auto &[bbArg, operand] :
          llvm::zip(payload.getOperands(), body->getArguments())) {
       if (bbArg != operand)