[Mlir-commits] [mlir] a786522 - [mlir][NFC] Fix codestyle issues introduced in D121988

[Markus Böck]

Author: Markus Böck
Date: 2022-03-28T16:55:01+02:00
New Revision: a7865228b30dc019c0d13d0c81247d90fcad325b

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

LOG: [mlir][NFC] Fix codestyle issues introduced in D121988

These were noticed in the post commit review of https://reviews.llvm.org/D121988

Added: 
    

Modified: 
    mlir/lib/Transforms/Utils/RegionUtils.cpp
    mlir/test/lib/Dialect/Test/TestDialect.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Transforms/Utils/RegionUtils.cpp b/mlir/lib/Transforms/Utils/RegionUtils.cpp
index 9d72be53f79ed..6ee3266f6a739 100644
--- a/mlir/lib/Transforms/Utils/RegionUtils.cpp
+++ b/mlir/lib/Transforms/Utils/RegionUtils.cpp
@@ -533,7 +533,7 @@ LogicalResult BlockMergeCluster::addToCluster(BlockEquivalenceData &blockData) {
 
         if (!isValidSuccessorArg(leaderBlock, lhsOperand) ||
             !isValidSuccessorArg(mergeBlock, rhsOperand))
-          return mlir::failure();
+          return failure();
 
         mismatchedOperands.emplace_back(opI, operand);
         continue;

diff  --git a/mlir/test/lib/Dialect/Test/TestDialect.cpp b/mlir/test/lib/Dialect/Test/TestDialect.cpp
index 7c106deeded0e..ebd21a0893321 100644
--- a/mlir/test/lib/Dialect/Test/TestDialect.cpp
+++ b/mlir/test/lib/Dialect/Test/TestDialect.cpp
@@ -348,9 +348,8 @@ TestBranchOp::getMutableSuccessorOperands(unsigned index) {
 Optional<MutableOperandRange>
 TestProducingBranchOp::getMutableSuccessorOperands(unsigned index) {
   assert(index <= 1 && "invalid successor index");
-  if (index == 1) {
+  if (index == 1)
     return getFirstOperandsMutable();
-  }
   return getSecondOperandsMutable();
 }