From aefc8fd41588360dd06cc4727efc5784bbcd3690 Mon Sep 17 00:00:00 2001 From: Oliver Hunt Date: Sun, 2 Dec 2007 00:11:25 +0000 Subject: [PATCH] Support initalisers for more than just int-typed static variables. We now use the CodeGenModule logic for generating the constant initialiser expression, so happily further initialiser fixes should automatically work for statics as well. llvm-svn: 44495 --- clang/CodeGen/CGDecl.cpp | 9 +++------ clang/CodeGen/CodeGenModule.cpp | 7 +++---- clang/CodeGen/CodeGenModule.h | 4 ++-- clang/test/CodeGen/staticinit.c | 10 ++++++++++ 4 files changed, 18 insertions(+), 12 deletions(-) diff --git a/clang/CodeGen/CGDecl.cpp b/clang/CodeGen/CGDecl.cpp index 09a47b9e5e6b..b9f072b65fa7 100644 --- a/clang/CodeGen/CGDecl.cpp +++ b/clang/CodeGen/CGDecl.cpp @@ -75,14 +75,11 @@ void CodeGenFunction::EmitStaticBlockVarDecl(const BlockVarDecl &D) { llvm::Constant *Init = 0; if (D.getInit() == 0) { Init = llvm::Constant::getNullValue(LTy); - } else if (D.getType()->isIntegerType()) { - llvm::APSInt Value(static_cast( - getContext().getTypeSize(D.getInit()->getType(), SourceLocation()))); - if (D.getInit()->isIntegerConstantExpr(Value, getContext())) - Init = llvm::ConstantInt::get(Value); + } else { + Init = CGM.EmitGlobalInit(D.getInit()); } - assert(Init && "FIXME: Support initializers"); + assert(Init && "Unable to create initialiser for static decl"); DMEntry = new llvm::GlobalVariable(LTy, false, diff --git a/clang/CodeGen/CodeGenModule.cpp b/clang/CodeGen/CodeGenModule.cpp index e4d0f47b0c3f..171a70c5a0cc 100644 --- a/clang/CodeGen/CodeGenModule.cpp +++ b/clang/CodeGen/CodeGenModule.cpp @@ -275,9 +275,8 @@ static llvm::Constant *GenerateConstantExpr(const Expr* Expression, return 0; } -llvm::Constant *CodeGenModule::EmitGlobalInit(const FileVarDecl *D, - llvm::GlobalVariable *GV) { - return GenerateConstantExpr(D->getInit(), *this); +llvm::Constant *CodeGenModule::EmitGlobalInit(const Expr *Expression) { + return GenerateConstantExpr(Expression, *this); } void CodeGenModule::EmitGlobalVar(const FileVarDecl *D) { @@ -300,7 +299,7 @@ void CodeGenModule::EmitGlobalVar(const FileVarDecl *D) { } if (!Init) - Init = EmitGlobalInit(D, GV); + Init = EmitGlobalInit(D->getInit()); assert(Init && "FIXME: Global variable initializers unimp!"); diff --git a/clang/CodeGen/CodeGenModule.h b/clang/CodeGen/CodeGenModule.h index 1d7c30e48729..9278e6864104 100644 --- a/clang/CodeGen/CodeGenModule.h +++ b/clang/CodeGen/CodeGenModule.h @@ -30,6 +30,7 @@ namespace clang { class ASTContext; class FunctionDecl; class Decl; + class Expr; class ValueDecl; class FileVarDecl; struct LangOptions; @@ -76,8 +77,7 @@ public: void EmitFunction(const FunctionDecl *FD); void EmitGlobalVar(const FileVarDecl *D); void EmitGlobalVarDeclarator(const FileVarDecl *D); - llvm::Constant *EmitGlobalInit(const FileVarDecl *D, - llvm::GlobalVariable *GV); + llvm::Constant *EmitGlobalInit(const Expr *Expression); void PrintStats() {} }; diff --git a/clang/test/CodeGen/staticinit.c b/clang/test/CodeGen/staticinit.c index 7411be32202c..e226179768ca 100644 --- a/clang/test/CodeGen/staticinit.c +++ b/clang/test/CodeGen/staticinit.c @@ -1,5 +1,15 @@ // RUN: clang -emit-llvm %s +struct AStruct { + int i; + char *s; + double d; +}; + void f() { static int i = 42; + static int is[] = { 1, 2, 3, 4 }; + static char* str = "forty-two"; + static char* strs[] = { "one", "two", "three", "four" }; + static struct AStruct myStruct = { 1, "two", 3.0 }; }