[llvm] r309606 - [SLPVectorizer] Unbreak the build with -Werror.

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 31 12:14:19 PDT 2017


Author: davide
Date: Mon Jul 31 12:14:19 2017
New Revision: 309606

URL: http://llvm.org/viewvc/llvm-project?rev=309606&view=rev
Log:
[SLPVectorizer] Unbreak the build with -Werror.

GCC was complaining about `&&` within `||` without explicit
parentheses. NFCI.

Modified:
    llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp

Modified: llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp?rev=309606&r1=309605&r2=309606&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp Mon Jul 31 12:14:19 2017
@@ -4385,9 +4385,9 @@ class HorizontalReduction {
     /// Checks if two operation data are both a reduction op or both a reduced
     /// value.
     bool operator==(const OperationData &OD) {
-      assert((IsReducedValue != OD.IsReducedValue) ||
-             ((!LHS == !OD.LHS) && (!RHS == !OD.RHS)) &&
-                 "One of the comparing operations is incorrect.");
+      assert(((IsReducedValue != OD.IsReducedValue) ||
+              ((!LHS == !OD.LHS) && (!RHS == !OD.RHS))) &&
+             "One of the comparing operations is incorrect.");
       return this == &OD ||
              (IsReducedValue == OD.IsReducedValue && Opcode == OD.Opcode);
     }




More information about the llvm-commits mailing list