From: Mikael Holmen <mikael.holmen@ericsson.com>
Date: Thu, 9 May 2019 12:11:57 +0000 (+0000)
Subject: Fix gcc compilation warning in an assert [NFC]
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e889472f6daf2da06925f0124a983e3f0e10cb61;p=clang

Fix gcc compilation warning in an assert [NFC]

Without this, gcc (7.4) complains with

../tools/clang/lib/Parse/ParseDecl.cpp:3937:63: error: suggest parentheses around '&&' within '||' [-Werror=parentheses]
     assert(!isAlreadyConsumed || RangeEnd != SourceLocation() &&
                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
                                      "both or neither of isAlreadyConsumed and "
                                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                      "RangeEnd needs to be set");
                                      ~

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@360333 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Parse/ParseDecl.cpp b/lib/Parse/ParseDecl.cpp
index 7b4ca124dc..bf13dd9611 100644
--- a/lib/Parse/ParseDecl.cpp
+++ b/lib/Parse/ParseDecl.cpp
@@ -3934,7 +3934,7 @@ void Parser::ParseDeclarationSpecifiers(DeclSpec &DS,
       continue;
     }
 
-    assert(!isAlreadyConsumed || RangeEnd != SourceLocation() &&
+    assert((!isAlreadyConsumed || RangeEnd != SourceLocation()) &&
                                      "both or neither of isAlreadyConsumed and "
                                      "RangeEnd needs to be set");
     DS.SetRangeEnd(isAlreadyConsumed ? RangeEnd : Tok.getLocation());