[llvm] r258619 - Inline variable into assert

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 22 22:49:30 PST 2016


Author: matze
Date: Sat Jan 23 00:49:29 2016
New Revision: 258619

URL: http://llvm.org/viewvc/llvm-project?rev=258619&view=rev
Log:
Inline variable into assert

Seems like some compilers still give unused variable warnings for
bool var = ...;
(void)var;
so I have to inline the variable.

Modified:
    llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp

Modified: llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp?rev=258619&r1=258618&r2=258619&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp Sat Jan 23 00:49:29 2016
@@ -1446,10 +1446,8 @@ static SDValue emitConjunctionDisjunctio
       std::swap(LHS, RHS);
   } else {
     bool NeedsNegOutL = LHS->getOpcode() == ISD::OR;
-    bool NeedsNegOutR = RHS->getOpcode() == ISD::OR;
-    assert((!NeedsNegOutR || !NeedsNegOutL) &&
+    assert((!NeedsNegOutL || RHS->getOpcode() != ISD::OR) &&
            "Valid conjunction/disjunction tree");
-    (void)NeedsNegOutR;
     // Order the side where we need to negate the output flags to RHS so it
     // gets emitted first.
     if (NeedsNegOutL)




More information about the llvm-commits mailing list