Apply clang-tidy fixes for llvm-else-after-return in ElementwiseOpFusion.cpp (NFC)

joker-eph · joker-eph · commit 46ce993dd414 · 2024-01-19T17:58:14.000-08:00
diff --git a/mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp b/mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp
@@ -1403,11 +1403,10 @@ static Value getCollapsedOpOperand(Location loc, LinalgOp op,
     return builder
         .create<memref::CollapseShapeOp>(loc, operand, operandReassociation)
         .getResult();
-  } else {
-    return builder
-        .create<tensor::CollapseShapeOp>(loc, operand, operandReassociation)
-        .getResult();
   }
+  return builder
+      .create<tensor::CollapseShapeOp>(loc, operand, operandReassociation)
+      .getResult();
 }
 
 /// Modify the `linalg.index` operations in the original generic op, to its

Original file line number	Diff line number	Diff line change
`@@ -1403,11 +1403,10 @@ static Value getCollapsedOpOperand(Location loc, LinalgOp op,`
`1403`	`1403`	`return builder`
`1404`	`1404`	`.create<memref::CollapseShapeOp>(loc, operand, operandReassociation)`
`1405`	`1405`	`.getResult();`
`1406`		`- } else {`
`1407`		`- return builder`
`1408`		`- .create<tensor::CollapseShapeOp>(loc, operand, operandReassociation)`
`1409`		`- .getResult();`
`1410`	`1406`	`}`
	`1407`	`+ return builder`
	`1408`	`+ .create<tensor::CollapseShapeOp>(loc, operand, operandReassociation)`
	`1409`	`+ .getResult();`
`1411`	`1410`	`}`
`1412`	`1411`
`1413`	`1412`	/// Modify the `linalg.index` operations in the original generic op, to its