[clang-tools-extra] 7180d95 - Fix MSVC "not all control paths return a value" warning. NFCI.

Simon Pilgrim via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 6 02:21:54 PST 2020


Author: Simon Pilgrim
Date: 2020-01-06T10:20:20Z
New Revision: 7180d9568df7d9198a75cfb5a156d9e60329794a

URL: https://github.com/llvm/llvm-project/commit/7180d9568df7d9198a75cfb5a156d9e60329794a
DIFF: https://github.com/llvm/llvm-project/commit/7180d9568df7d9198a75cfb5a156d9e60329794a.diff

LOG: Fix MSVC "not all control paths return a value" warning. NFCI.

Added: 
    

Modified: 
    clang-tools-extra/clang-tidy/utils/FixItHintUtils.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clang-tidy/utils/FixItHintUtils.cpp b/clang-tools-extra/clang-tidy/utils/FixItHintUtils.cpp
index a9c95e9f3810..dd7c24d0c7ef 100644
--- a/clang-tools-extra/clang-tidy/utils/FixItHintUtils.cpp
+++ b/clang-tools-extra/clang-tidy/utils/FixItHintUtils.cpp
@@ -95,6 +95,7 @@ static Optional<FixItHint> changeValue(const VarDecl &Var,
       return fixIfNotDangerous(*IgnoredParens, buildQualifier(Qualifier));
     return None;
   }
+  llvm_unreachable("Unknown QualifierPolicy enum");
 }
 
 static Optional<FixItHint> changePointerItself(const VarDecl &Var,


        


More information about the cfe-commits mailing list