diff options
| author | Douglas Gregor <dgregor@apple.com> | 2010-05-05 20:15:55 +0000 |
|---|---|---|
| committer | Douglas Gregor <dgregor@apple.com> | 2010-05-05 20:15:55 +0000 |
| commit | d450f06ef4243e45fb078edeffcccc21915c1247 (patch) | |
| tree | 0eeac89de49ba7ea79dd4c97264ede65bc0d20aa | |
| parent | e36cd72e38e1a742c79f730f968bb4f8f16f33eb (diff) | |
| download | bcm5719-llvm-d450f06ef4243e45fb078edeffcccc21915c1247.tar.gz bcm5719-llvm-d450f06ef4243e45fb078edeffcccc21915c1247.zip | |
When we emit a non-constant initializer for a global variable of
reference type, make sure that the initializer we build is the
of the appropriate type for the *reference*, not for the thing that it
refers to. Fixes PR7050.
llvm-svn: 103115
| -rw-r--r-- | clang/lib/CodeGen/CodeGenModule.cpp | 6 | ||||
| -rw-r--r-- | clang/test/CodeGenCXX/static-init-3.cpp | 28 |
2 files changed, 32 insertions, 2 deletions
diff --git a/clang/lib/CodeGen/CodeGenModule.cpp b/clang/lib/CodeGen/CodeGenModule.cpp index cc90a2855de..9c84bcec054 100644 --- a/clang/lib/CodeGen/CodeGenModule.cpp +++ b/clang/lib/CodeGen/CodeGenModule.cpp @@ -1101,10 +1101,12 @@ void CodeGenModule::EmitGlobalVarDefinition(const VarDecl *D) { assert(!ASTTy->isIncompleteType() && "Unexpected incomplete type"); Init = EmitNullConstant(D->getType()); } else { - Init = EmitConstantExpr(InitExpr, D->getType()); - + Init = EmitConstantExpr(InitExpr, D->getType()); if (!Init) { QualType T = InitExpr->getType(); + if (D->getType()->isReferenceType()) + T = D->getType(); + if (getLangOptions().CPlusPlus) { EmitCXXGlobalVarDeclInitFunc(D); Init = EmitNullConstant(T); diff --git a/clang/test/CodeGenCXX/static-init-3.cpp b/clang/test/CodeGenCXX/static-init-3.cpp new file mode 100644 index 00000000000..5bf76a61708 --- /dev/null +++ b/clang/test/CodeGenCXX/static-init-3.cpp @@ -0,0 +1,28 @@ +// RUN: %clang_cc1 -emit-llvm -triple x86_64-apple-darwin10.0.0 -o - %s | FileCheck %s + +// PR7050 +template<class T> struct X0 : public T { }; + +template <class T> +struct X1 +{ + static T & instance; + // include this to provoke instantiation at pre-execution time + static void use(T const &) {} + static T & get() { + static X0<T> t; + use(instance); + return static_cast<T &>(t); + } +}; + +// CHECK: @_ZN2X1I2X2I1BEE8instanceE = weak global %struct.X0* null, align 8 +// CHECJ: @_ZN2X1I2X2I1AEE8instanceE = weak global %struct.X0* null, align 8 +template<class T> T & X1<T>::instance = X1<T>::get(); + +class A { }; +class B : public A { }; + +template<typename T> struct X2 {}; +X2< B > bg = X1< X2< B > >::get(); +X2< A > ag = X1< X2< A > >::get(); |

