]> granicus.if.org Git - clang/commitdiff
Fix one more 'not all control paths return a value' MSVC warning
authorAlex Lorenz <arphaman@gmail.com>
Tue, 18 Apr 2017 10:17:41 +0000 (10:17 +0000)
committerAlex Lorenz <arphaman@gmail.com>
Tue, 18 Apr 2017 10:17:41 +0000 (10:17 +0000)
The warning was caused by r300539.

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

lib/Parse/ParsePragma.cpp
utils/TableGen/ClangAttrEmitter.cpp

index aa915fa3ba1cb189aca45872c028e818b3a097fd..abddb6ba6704941d9e313f2e32412f217b3b83b3 100644 (file)
@@ -1014,6 +1014,8 @@ static bool isAbstractAttrMatcherRule(attr::SubjectMatchRule Rule) {
     return IsAbstract;
 #include "clang/Basic/AttrSubMatchRulesList.inc"
   }
+  llvm_unreachable("Invalid attribute subject match rule");
+  return false;
 }
 
 static void diagnoseExpectedAttributeSubjectSubRule(
index 7aa8bdc781e831b2115cca65615ff96638f6779c..65dbccde62c37b8f6fb8f94cbc445f7e937f2d73 100644 (file)
@@ -3168,7 +3168,7 @@ emitAttributeMatchRules(PragmaClangAttributeSupport &PragmaAttributeSupport,
     OS << ";\n";
   }
   OS << "  }\n";
-  OS << "  llvm_unreachable(\"Invalid match rule\");\n";
+  OS << "  llvm_unreachable(\"Invalid match rule\");\nreturn false;\n";
   OS << "}\n\n";
 }