From 74821754e7cf57c4295b12852fdb8b748cf41760 Mon Sep 17 00:00:00 2001 From: Mehdi Amini Date: Thu, 17 Nov 2022 10:40:56 +0000 Subject: [PATCH] Apply clang-tidy fixes for readability-simplify-boolean-expr in CSE.cpp (NFC) --- mlir/lib/Transforms/CSE.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mlir/lib/Transforms/CSE.cpp b/mlir/lib/Transforms/CSE.cpp index 97f6cfd..5e5852a 100644 --- a/mlir/lib/Transforms/CSE.cpp +++ b/mlir/lib/Transforms/CSE.cpp @@ -263,8 +263,8 @@ LogicalResult CSE::simplifyOperation(ScopedMapTy &knownValues, Operation *op, // Don't simplify operations with nested blocks. We don't currently model // equality comparisons correctly among other things. It is also unclear // whether we would want to CSE such operations. - if (!(op->getNumRegions() == 0 || - (op->getNumRegions() == 1 && llvm::hasSingleElement(op->getRegion(0))))) + if (op->getNumRegions() != 0 && + (op->getNumRegions() != 1 || !llvm::hasSingleElement(op->getRegion(0)))) return failure(); // Some simple use case of operation with memory side-effect are dealt with -- 2.7.4