Apply clang-tidy fixes for llvm-else-after-return in SCF.cpp (NFC)
authorMehdi Amini <joker.eph@gmail.com>
Sun, 3 Apr 2022 23:18:19 +0000 (23:18 +0000)
committerMehdi Amini <joker.eph@gmail.com>
Wed, 13 Apr 2022 23:14:44 +0000 (23:14 +0000)
mlir/lib/Dialect/SCF/SCF.cpp

index 3e5028f..f010bc6 100644 (file)
@@ -1793,9 +1793,8 @@ struct CombineNestedIfs : public OpRewritePattern<IfOp> {
       // If the then value is defined within the scf.if, bail.
       if (tup.value().getParentRegion() == &op.getThenRegion()) {
         return failure();
-      } else {
-        elseYieldsToUpgradeToSelect.push_back(tup.index());
       }
+      elseYieldsToUpgradeToSelect.push_back(tup.index());
     }
 
     Location loc = op.getLoc();