summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Blaikie <dblaikie@gmail.com>2019-05-03 18:11:31 +0000
committerDavid Blaikie <dblaikie@gmail.com>2019-05-03 18:11:31 +0000
commit1801280d2dcb52a20c177f780cd4e1d37d0b5eb3 (patch)
treea23c334d0ec4f42426b3af4412a67f3ec1deab8f
parent964c2356512fb31b502b5d200a9f2716c93a3805 (diff)
Remove else-after-return
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@359913 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--lib/AST/ExprConstant.cpp9
1 files changed, 4 insertions, 5 deletions
diff --git a/lib/AST/ExprConstant.cpp b/lib/AST/ExprConstant.cpp
index 8386ce8f2d..6b3f4dcefa 100644
--- a/lib/AST/ExprConstant.cpp
+++ b/lib/AST/ExprConstant.cpp
@@ -8269,17 +8269,16 @@ bool IntExprEvaluator::VisitBuiltinCallExpr(const CallExpr *E,
case Builtin::BI__builtin_constant_p: {
const Expr *Arg = E->getArg(0);
- if (EvaluateBuiltinConstantP(Info, Arg)) {
+ if (EvaluateBuiltinConstantP(Info, Arg))
return Success(true, E);
- } else if (Info.InConstantContext || Arg->HasSideEffects(Info.Ctx)) {
+ if (Info.InConstantContext || Arg->HasSideEffects(Info.Ctx)) {
// Outside a constant context, eagerly evaluate to false in the presence
// of side-effects in order to avoid -Wunsequenced false-positives in
// a branch on __builtin_constant_p(expr).
return Success(false, E);
- } else {
- Info.FFDiag(E, diag::note_invalid_subexpr_in_const_expr);
- return false;
}
+ Info.FFDiag(E, diag::note_invalid_subexpr_in_const_expr);
+ return false;
}
case Builtin::BI__builtin_is_constant_evaluated: