]> granicus.if.org Git - clang/commitdiff
Placate -Wunreachable-code by removing unnecessary logic to handle NUM_OPENMP_DEFAULT...
authorTed Kremenek <kremenek@apple.com>
Fri, 21 Mar 2014 17:34:28 +0000 (17:34 +0000)
committerTed Kremenek <kremenek@apple.com>
Fri, 21 Mar 2014 17:34:28 +0000 (17:34 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@204487 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaOpenMP.cpp

index e43b4f453ec2c21ee446cb1aff9311d34902af0f..dc2a68d418dadf57e5e1b828b6fa4d600a7ca23b 100644 (file)
@@ -937,7 +937,9 @@ OMPClause *Sema::ActOnOpenMPDefaultClause(OpenMPDefaultClauseKind Kind,
                                           SourceLocation EndLoc) {
   if (Kind == OMPC_DEFAULT_unknown) {
     std::string Values;
-    std::string Sep(NUM_OPENMP_DEFAULT_KINDS > 1 ? ", " : "");
+    static_assert(NUM_OPENMP_DEFAULT_KINDS > 1,
+                  "NUM_OPENMP_DEFAULT_KINDS not greater than 1");
+    std::string Sep(", ");
     for (unsigned i = OMPC_DEFAULT_unknown + 1;
          i < NUM_OPENMP_DEFAULT_KINDS; ++i) {
       Values += "'";