From: Anders Carlsson Date: Wed, 8 Apr 2009 04:48:15 +0000 (+0000) Subject: Add a destination type argument to EmitConstantExpr. This will be used for when the... X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e9352cc9818ba59e7cf88500ef048991c90f3821;p=clang Add a destination type argument to EmitConstantExpr. This will be used for when the destination has a reference type. (No functionality change yet) git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@68593 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/CodeGen/CGBuiltin.cpp b/lib/CodeGen/CGBuiltin.cpp index 0b40ec76e8..8767eb4fb3 100644 --- a/lib/CodeGen/CGBuiltin.cpp +++ b/lib/CodeGen/CGBuiltin.cpp @@ -66,7 +66,7 @@ RValue CodeGenFunction::EmitBuiltinExpr(const FunctionDecl *FD, switch (BuiltinID) { default: break; // Handle intrinsics and libm functions below. case Builtin::BI__builtin___CFStringMakeConstantString: - return RValue::get(CGM.EmitConstantExpr(E, 0)); + return RValue::get(CGM.EmitConstantExpr(E, E->getType(), 0)); case Builtin::BI__builtin_stdarg_start: case Builtin::BI__builtin_va_start: case Builtin::BI__builtin_va_end: { diff --git a/lib/CodeGen/CGDecl.cpp b/lib/CodeGen/CGDecl.cpp index b8f5ca85cf..f0822675f8 100644 --- a/lib/CodeGen/CGDecl.cpp +++ b/lib/CodeGen/CGDecl.cpp @@ -121,7 +121,7 @@ void CodeGenFunction::EmitStaticBlockVarDecl(const VarDecl &D) { DMEntry = GV; if (D.getInit()) { - llvm::Constant *Init = CGM.EmitConstantExpr(D.getInit(), this); + llvm::Constant *Init = CGM.EmitConstantExpr(D.getInit(), D.getType(), this); // If constant emission failed, then this should be a C++ static // initializer. diff --git a/lib/CodeGen/CGExprConstant.cpp b/lib/CodeGen/CGExprConstant.cpp index 51ad5b67b8..35b3a3750a 100644 --- a/lib/CodeGen/CGExprConstant.cpp +++ b/lib/CodeGen/CGExprConstant.cpp @@ -55,7 +55,9 @@ public: // GCC cast to union extension if (E->getType()->isUnionType()) { const llvm::Type *Ty = ConvertType(E->getType()); - return EmitUnion(CGM.EmitConstantExpr(E->getSubExpr(), CGF), Ty); + Expr *SubExpr = E->getSubExpr(); + return EmitUnion(CGM.EmitConstantExpr(SubExpr, SubExpr->getType(), CGF), + Ty); } if (CGM.getContext().getCanonicalType(E->getSubExpr()->getType()) == CGM.getContext().getCanonicalType(E->getType())) { @@ -91,7 +93,8 @@ public: unsigned i = 0; bool RewriteType = false; for (; i < NumInitableElts; ++i) { - llvm::Constant *C = CGM.EmitConstantExpr(ILE->getInit(i), CGF); + Expr *Init = ILE->getInit(i); + llvm::Constant *C = CGM.EmitConstantExpr(Init, Init->getType(), CGF); if (!C) return 0; RewriteType |= (C->getType() != ElemTy); @@ -117,7 +120,7 @@ public: void InsertBitfieldIntoStruct(std::vector& Elts, FieldDecl* Field, Expr* E) { // Calculate the value to insert - llvm::Constant *C = CGM.EmitConstantExpr(E, CGF); + llvm::Constant *C = CGM.EmitConstantExpr(E, Field->getType(), CGF); if (!C) return; @@ -204,7 +207,8 @@ public: InsertBitfieldIntoStruct(Elts, *Field, ILE->getInit(EltNo)); } else { unsigned FieldNo = CGM.getTypes().getLLVMFieldNo(*Field); - llvm::Constant *C = CGM.EmitConstantExpr(ILE->getInit(EltNo), CGF); + llvm::Constant *C = CGM.EmitConstantExpr(ILE->getInit(EltNo), + Field->getType(), CGF); if (!C) return 0; RewriteType |= (C->getType() != Elts[FieldNo]->getType()); Elts[FieldNo] = C; @@ -283,7 +287,8 @@ public: return llvm::ConstantArray::get(RetTy, Elts); } - return EmitUnion(CGM.EmitConstantExpr(ILE->getInit(0), CGF), Ty); + Expr *Init = ILE->getInit(0); + return EmitUnion(CGM.EmitConstantExpr(Init, Init->getType(), CGF), Ty); } llvm::Constant *EmitVectorInitialization(InitListExpr *ILE) { @@ -299,7 +304,8 @@ public: // Copy initializer elements. unsigned i = 0; for (; i < NumInitableElts; ++i) { - llvm::Constant *C = CGM.EmitConstantExpr(ILE->getInit(i), CGF); + Expr *Init = ILE->getInit(i); + llvm::Constant *C = CGM.EmitConstantExpr(Init, Init->getType(), CGF); if (!C) return 0; Elts.push_back(C); @@ -319,9 +325,10 @@ public: llvm::Constant *VisitInitListExpr(InitListExpr *ILE) { if (ILE->getType()->isScalarType()) { // We have a scalar in braces. Just use the first element. - if (ILE->getNumInits() > 0) - return CGM.EmitConstantExpr(ILE->getInit(0), CGF); - + if (ILE->getNumInits() > 0) { + Expr *Init = ILE->getInit(0); + return CGM.EmitConstantExpr(Init, Init->getType(), CGF); + } const llvm::Type* RetTy = CGM.getTypes().ConvertType(ILE->getType()); return llvm::Constant::getNullValue(RetTy); } @@ -463,6 +470,7 @@ public: } // end anonymous namespace. llvm::Constant *CodeGenModule::EmitConstantExpr(const Expr *E, + QualType DestType, CodeGenFunction *CGF) { Expr::EvalResult Result; diff --git a/lib/CodeGen/CodeGenModule.cpp b/lib/CodeGen/CodeGenModule.cpp index 508d7ce2b6..2c1ab1f8b5 100644 --- a/lib/CodeGen/CodeGenModule.cpp +++ b/lib/CodeGen/CodeGenModule.cpp @@ -693,7 +693,7 @@ void CodeGenModule::EmitGlobalVarDefinition(const VarDecl *D) { } Init = llvm::Constant::getNullValue(InitTy); } else { - Init = EmitConstantExpr(D->getInit()); + Init = EmitConstantExpr(D->getInit(), D->getType()); if (!Init) { ErrorUnsupported(D, "static initializer"); QualType T = D->getInit()->getType(); diff --git a/lib/CodeGen/CodeGenModule.h b/lib/CodeGen/CodeGenModule.h index 8661d23caf..71cfb64098 100644 --- a/lib/CodeGen/CodeGenModule.h +++ b/lib/CodeGen/CodeGenModule.h @@ -256,7 +256,8 @@ public: /// EmitConstantExpr - Try to emit the given expression as a /// constant; returns 0 if the expression cannot be emitted as a /// constant. - llvm::Constant *EmitConstantExpr(const Expr *E, CodeGenFunction *CGF = 0); + llvm::Constant *EmitConstantExpr(const Expr *E, QualType DestType, + CodeGenFunction *CGF = 0); llvm::Constant *EmitAnnotateAttr(llvm::GlobalValue *GV, const AnnotateAttr *AA, unsigned LineNo);