[Mlir-commits] [mlir] c067a9d - Apply clang-tidy fixes for llvm-else-after-return in OpenMPDialect.cpp (NFC)

Mehdi Amini llvmlistbot at llvm.org
Thu May 26 14:30:33 PDT 2022


Author: Mehdi Amini
Date: 2022-05-26T21:29:58Z
New Revision: c067a9dee5c1c115f6b11ec12f5719613ef08391

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

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

Added: 
    

Modified: 
    mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp b/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
index 23a25fc4260b8..19615c06b08dc 100644
--- a/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
+++ b/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
@@ -985,7 +985,8 @@ LogicalResult CancelOp::verify() {
     if (cast<WsLoopOp>(parentOp).nowaitAttr()) {
       return emitError() << "A worksharing construct that is canceled "
                          << "must not have a nowait clause";
-    } else if (cast<WsLoopOp>(parentOp).ordered_valAttr()) {
+    }
+    if (cast<WsLoopOp>(parentOp).ordered_valAttr()) {
       return emitError() << "A worksharing construct that is canceled "
                          << "must not have an ordered clause";
     }


        


More information about the Mlir-commits mailing list