[cfe-commits] r106403 - /cfe/trunk/lib/Checker/SimpleSValuator.cpp

[Benjamin Kramer]

Author: d0k
Date: Sun Jun 20 05:20:36 2010
New Revision: 106403

URL: http://llvm.org/viewvc/llvm-project?rev=106403&view=rev
Log:
Add braces to avoid an ambiguous else, fixing a GCC warning.

Modified:
    cfe/trunk/lib/Checker/SimpleSValuator.cpp

Modified: cfe/trunk/lib/Checker/SimpleSValuator.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Checker/SimpleSValuator.cpp?rev=106403&r1=106402&r2=106403&view=diff
==============================================================================
--- cfe/trunk/lib/Checker/SimpleSValuator.cpp (original)
+++ cfe/trunk/lib/Checker/SimpleSValuator.cpp Sun Jun 20 05:20:36 2010
@@ -278,11 +278,11 @@
 
   // 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.