Add parentheses around boolean operators in assert
authorAlex Zinenko <zinenko@google.com>
Thu, 3 Oct 2019 08:46:39 +0000 (01:46 -0700)
committerA. Unique TensorFlower <gardener@tensorflow.org>
Thu, 3 Oct 2019 08:47:14 +0000 (01:47 -0700)
This removes a warning and is generally a good practice.

PiperOrigin-RevId: 272613597

mlir/lib/Conversion/StandardToLLVM/ConvertStandardToLLVM.cpp

index a361aaa..a5137ef 100644 (file)
@@ -659,8 +659,9 @@ struct AllocOpLowering : public LLVMLegalizationPattern<AllocOp> {
            "unexpected dynamic offset");
 
     // 0-D memref corner case: they have size 1 ...
-    assert((type.getRank() == 0 && strides.empty() && sizes.size() == 1) ||
-           (strides.size() == sizes.size()) && "unexpected number of strides");
+    assert(((type.getRank() == 0 && strides.empty() && sizes.size() == 1) ||
+            (strides.size() == sizes.size())) &&
+           "unexpected number of strides");
 
     // Create the MemRef descriptor.
     auto structType = lowering.convertType(type);