[mlir][sparse] fix switch statement bug on two binary ops

[aartbik]

Summary

They appeared at the wrong place in the switch, treating
them as unary op rather than binary op.

[aartbik]

rebased against main

mlir/unittests/Dialect/SparseTensor/MergerTest.cpp

bool compareExpression(unsigned e, const std::shared_ptr<Pattern> &pattern) {
     case kCastU:
     case kCastIdx:
     case kTruncI:
     case kCIm:
     case kCRe:
     case kBitCast:
     case kBinaryBranch:
     case kUnary:
-    case kShlI:
-    case kBinary:
       return compareExpression(tensorExp.children.e0, pattern->e0);
     // Binary operations.
     case kMulF:
     case kMulC:
     case kMulI:
     case kDivF:
     case kDivC:
     case kDivS:
     case kDivU:
     case kAddF:
     case kAddC:
     case kAddI:
     case kSubF:
     case kSubC:
     case kSubI:
     case kAndI:
     case kOrI:
     case kXorI:
     case kShrS:
     case kShrU:
+    case kShlI:
+    case kBinary:
       return compareExpression(tensorExp.children.e0, pattern->e0) &&
              compareExpression(tensorExp.children.e1, pattern->e1);
     }
     llvm_unreachable("unexpected kind");
   }
 
   unsigned numTensors;
   unsigned numLoops;