]> granicus.if.org Git - clang/commitdiff
Fix "enumeral mismatch in conditional expression" gcc7 warnings. NFCI.
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 17 Dec 2018 12:17:37 +0000 (12:17 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 17 Dec 2018 12:17:37 +0000 (12:17 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@349342 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaDeclAttr.cpp
lib/Sema/SemaDeclCXX.cpp

index 78374b80894f562ff4490c51526fb5abd08703f3..ba041a8786b985e2f094abae45c572ddf8f1a28b 100644 (file)
@@ -6103,9 +6103,10 @@ static void ProcessDeclAttribute(Sema &S, Scope *scope, Decl *D,
   // though they were unknown attributes.
   if (AL.getKind() == ParsedAttr::UnknownAttribute ||
       !AL.existsInTarget(S.Context.getTargetInfo())) {
-    S.Diag(AL.getLoc(), AL.isDeclspecAttribute()
-                            ? diag::warn_unhandled_ms_attribute_ignored
-                            : diag::warn_unknown_attribute_ignored)
+    S.Diag(AL.getLoc(),
+           AL.isDeclspecAttribute()
+               ? (unsigned)diag::warn_unhandled_ms_attribute_ignored
+               : (unsigned)diag::warn_unknown_attribute_ignored)
         << AL;
     return;
   }
index d56c880311393ccfa62318fbdfc3452059c8e83c..d46aa23c274b4d253f6faca3d9ceb8d1aa1c5b3e 100644 (file)
@@ -2347,8 +2347,8 @@ Sema::ActOnBaseSpecifier(Decl *classdecl, SourceRange SpecifierRange,
     if (AL.isInvalid() || AL.getKind() == ParsedAttr::IgnoredAttribute)
       continue;
     Diag(AL.getLoc(), AL.getKind() == ParsedAttr::UnknownAttribute
-                          ? diag::warn_unknown_attribute_ignored
-                          : diag::err_base_specifier_attribute)
+                          ? (unsigned)diag::warn_unknown_attribute_ignored
+                          : (unsigned)diag::err_base_specifier_attribute)
         << AL.getName();
   }
 
@@ -2867,14 +2867,14 @@ static const ParsedAttr *getMSPropertyAttr(const ParsedAttributesView &list) {
   return nullptr;
 }
 
-// Check if there is a field shadowing.\r
-void Sema::CheckShadowInheritedFields(const SourceLocation &Loc,\r
-                                      DeclarationName FieldName,\r
-                                      const CXXRecordDecl *RD,\r
-                                      bool DeclIsField) {\r
-  if (Diags.isIgnored(diag::warn_shadow_field, Loc))\r
-    return;\r
-\r
+// Check if there is a field shadowing.
+void Sema::CheckShadowInheritedFields(const SourceLocation &Loc,
+                                      DeclarationName FieldName,
+                                      const CXXRecordDecl *RD,
+                                      bool DeclIsField) {
+  if (Diags.isIgnored(diag::warn_shadow_field, Loc))
+    return;
+
   // To record a shadowed field in a base
   std::map<CXXRecordDecl*, NamedDecl*> Bases;
   auto FieldShadowed = [&](const CXXBaseSpecifier *Specifier,
@@ -2908,13 +2908,13 @@ void Sema::CheckShadowInheritedFields(const SourceLocation &Loc,
       continue;
     auto BaseField = It->second;
     assert(BaseField->getAccess() != AS_private);
-    if (AS_none !=\r
-        CXXRecordDecl::MergeAccess(P.Access, BaseField->getAccess())) {\r
-      Diag(Loc, diag::warn_shadow_field)\r
-        << FieldName << RD << Base << DeclIsField;\r
-      Diag(BaseField->getLocation(), diag::note_shadow_field);\r
-      Bases.erase(It);\r
-    }\r
+    if (AS_none !=
+        CXXRecordDecl::MergeAccess(P.Access, BaseField->getAccess())) {
+      Diag(Loc, diag::warn_shadow_field)
+        << FieldName << RD << Base << DeclIsField;
+      Diag(BaseField->getLocation(), diag::note_shadow_field);
+      Bases.erase(It);
+    }
   }
 }