commit 0128f668a912fe06121a58d38f515e629e649ae5
author Benjamin Kramer <benny.kra@googlemail.com> Sun Jan 03 18:18:37 2010 +0000
committer Benjamin Kramer <benny.kra@googlemail.com> Sun Jan 03 18:18:37 2010 +0000
tree 4aa6dd39934e439a3a5348feaadace961fe17065
parent fd11f49b4e320c1fb43b99c42235273675d32642

__builtin_object_size(ptr, type) returns -1 for type = {0,1} if there are any side-effects.

llvm-svn: 92453

clang/lib/AST/ExprConstant.cpp

diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp
index 06afec7..d49a821 100644
--- a/clang/lib/AST/ExprConstant.cpp
+++ b/clang/lib/AST/ExprConstant.cpp
@@ -989,7 +989,7 @@
 
     // TODO: Perhaps we should let LLVM lower this?
     if (E->getArg(0)->HasSideEffects(Info.Ctx)) {
-      if (E->getArg(1)->EvaluateAsInt(Info.Ctx).getZExtValue() == 0)
+      if (E->getArg(1)->EvaluateAsInt(Info.Ctx).getZExtValue() <= 1)
         return Success(-1ULL, E);
       return Success(0, E);
     }