[cfe-commits] r62723 - /cfe/trunk/lib/Sema/Sema.h

Steve Naroff snaroff at apple.com
Wed Jan 21 14:32:33 PST 2009


Author: snaroff
Date: Wed Jan 21 16:32:33 2009
New Revision: 62723

URL: http://llvm.org/viewvc/llvm-project?rev=62723&view=rev
Log:
Fix Sema::Owned(ExprResult) to not use a ternary operator. Necessary to work around a Visual Studio compiler bug.

Thanks to Doug Gregor for the suggestion.

Modified:
    cfe/trunk/lib/Sema/Sema.h

Modified: cfe/trunk/lib/Sema/Sema.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/Sema.h?rev=62723&r1=62722&r2=62723&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/Sema.h (original)
+++ cfe/trunk/lib/Sema/Sema.h Wed Jan 21 16:32:33 2009
@@ -249,7 +249,9 @@
 
   OwningExprResult Owned(Expr* E) { return OwningExprResult(*this, E); }
   OwningExprResult Owned(ExprResult R) {
-    return R.isInvalid ? ExprError() : OwningExprResult(*this, R.Val);
+    if (R.isInvalid)
+      return ExprError();
+    return OwningExprResult(*this, R.Val);
   }
   OwningStmtResult Owned(Stmt* S) { return OwningStmtResult(*this, S); }
 





More information about the cfe-commits mailing list