From: Eli Friedman <eli.friedman@gmail.com>
Date: Fri, 16 May 2008 13:28:37 +0000 (+0000)
Subject: Minor cleanup to isBuiltinConstantExpr.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=861dc46e3c53d64d8685fd9a0f95faac710b61c6;p=clang

Minor cleanup to isBuiltinConstantExpr.



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

diff --git a/lib/AST/Expr.cpp b/lib/AST/Expr.cpp
index 962819b733..cd495bc63e 100644
--- a/lib/AST/Expr.cpp
+++ b/lib/AST/Expr.cpp
@@ -141,9 +141,8 @@ bool CallExpr::isBuiltinConstantExpr() const {
     return false;
 
   // We have a builtin that is a constant expression
-  if (builtinID == Builtin::BI__builtin___CFStringMakeConstantString)
-    return true;
-  return false;
+  return builtinID == Builtin::BI__builtin___CFStringMakeConstantString ||
+         builtinID == Builtin::BI__builtin_classify_type;
 }
 
 bool CallExpr::isBuiltinClassifyType(llvm::APSInt &Result) const {
@@ -517,10 +516,6 @@ bool Expr::isConstantExpr(ASTContext &Ctx, SourceLocation *Loc) const {
     return true;
   case CallExprClass: {
     const CallExpr *CE = cast<CallExpr>(this);
-    llvm::APSInt Result(32);
-    Result.zextOrTrunc(static_cast<uint32_t>(Ctx.getTypeSize(getType())));
-    if (CE->isBuiltinClassifyType(Result))
-      return true;
     if (CE->isBuiltinConstantExpr())
       return true;
     if (Loc) *Loc = getLocStart();