[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