From: Benjamin Kramer <benny.kra@googlemail.com>
Date: Sun, 20 Jun 2010 10:20:36 +0000 (+0000)
Subject: Add braces to avoid an ambiguous else, fixing a GCC warning.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=24ada872d20b4ba14121f5246a9463c6c541a3a4;p=clang

Add braces to avoid an ambiguous else, fixing a GCC warning.


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

diff --git a/lib/Checker/SimpleSValuator.cpp b/lib/Checker/SimpleSValuator.cpp
index fe82276faa..0799380e80 100644
--- a/lib/Checker/SimpleSValuator.cpp
+++ b/lib/Checker/SimpleSValuator.cpp
@@ -278,11 +278,11 @@ SVal SimpleSValuator::MakeSymIntVal(const SymExpr *LHS,
 
   // Idempotent ops (like a*1) can still change the type of an expression.
   // Wrap the LHS up in a NonLoc again and let EvalCastNL do the dirty work.
-  if (isIdempotent)
+  if (isIdempotent) {
     if (SymbolRef LHSSym = dyn_cast<SymbolData>(LHS))
       return EvalCastNL(nonloc::SymbolVal(LHSSym), resultTy);
-    else
-      return EvalCastNL(nonloc::SymExprVal(LHS), resultTy);
+    return EvalCastNL(nonloc::SymExprVal(LHS), resultTy);
+  }
 
   // If we reach this point, the expression cannot be simplified.
   // Make a SymExprVal for the entire thing.