[Sema] Fix -Wparentheses warning seen with gcc

This patch simply replace assert(X || Y && "...") by assert((X || Y) && "...") to silence -Wparentheses warnings.
llvm · Aug 23, 2023 · f672094 · f672094
1 parent a23e01a
commit f672094
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
@@ -14379,10 +14379,9 @@ void Sema::CheckCompleteVariableDeclaration(VarDecl *var) {
       if (Stack != &ConstSegStack && MSVCEnv &&
           ConstSegStack.CurrentValue != ConstSegStack.DefaultValue &&
           var->getType().isConstQualified()) {
-        assert(!Reason ||
-               Reason != QualType::NonConstantStorageReason::
-                             NonConstNonReferenceType &&
-                   "This case should've already been handled elsewhere");
+        assert((!Reason || Reason != QualType::NonConstantStorageReason::
+                                         NonConstNonReferenceType) &&
+               "This case should've already been handled elsewhere");
         Diag(var->getLocation(), diag::warn_section_msvc_compat)
                 << var << ConstSegStack.CurrentValue << (int)(!HasConstInit
             ? QualType::NonConstantStorageReason::NonTrivialCtor