Apply review remarks

Author: ArtemySkrebkov

mlir/lib/Dialect/SCF/Transforms/StructuralTypeConversions.cpp

@@ -201,12 +201,11 @@ class ConvertIndexSwitchOpTypes
   convertSourceOp(IndexSwitchOp op, OneToNOpAdaptor adaptor,
                   ConversionPatternRewriter &rewriter,
                   TypeRange dstTypes) const {
-    auto newOp = rewriter.create<IndexSwitchOp>(
-        op.getLoc(), dstTypes, op.getArg(), op.getCases(), op.getNumCases());
+    auto newOp =
+        IndexSwitchOp::create(rewriter, op.getLoc(), dstTypes, op.getArg(),
+                              op.getCases(), op.getNumCases());
 
     for (unsigned i = 0u; i < op.getNumRegions(); i++) {
-      if (failed(rewriter.convertRegionTypes(&op.getRegion(i), *typeConverter)))
-        return std::nullopt;
       auto &dstRegion = newOp.getRegion(i);
       rewriter.inlineRegionBefore(op.getRegion(i), dstRegion, dstRegion.end());
     }

mlir/test/Dialect/SparseTensor/scf_1_N_conversion.mlir

@@ -103,6 +103,7 @@ func.func @while(%arg0: tensor<1024xf32, #SparseVector>, %c: i1) -> tensor<1024x
 // CHECK-SAME:                            %[[VAL_C_4:.*12]]: !sparse_tensor.storage_specifier
 
 // CHECK:           %[[RES:.*]]:4 = scf.index_switch %[[PRED]]
+// CHECK-SAME:          -> memref<?xindex>, memref<?xindex>, memref<?xf32>, !sparse_tensor.storage_specifier
 // CHECK:           case 1 {
 // CHECK:             scf.yield %[[VAL_A_1]], %[[VAL_A_2]], %[[VAL_A_3]], %[[VAL_A_4]]
 // CHECK:           case 2 {