commit f9d865b059a1fa11d9d8287e5ee60b8ad1a38b1e
author Hans Wennborg <hans@hanshq.net> Tue Mar 17 16:38:58 2015 +0000
committer Hans Wennborg <hans@hanshq.net> Tue Mar 17 16:38:58 2015 +0000
tree 5d343f38ce76c5209a8977d85c90b007a566e50d
parent 2eeace219dfb161d49a698becd6040f4d39059f3

Revert r232454 and r232456: "Fix the LLVM type used when lowering initializer list reference temporaries to global variables."

This caused PR22940.

llvm-svn: 232496

clang/lib/CodeGen/CGExpr.cpp

diff --git a/clang/lib/CodeGen/CGExpr.cpp b/clang/lib/CodeGen/CGExpr.cpp
index 9209875..35275e5 100644
--- a/clang/lib/CodeGen/CGExpr.cpp
+++ b/clang/lib/CodeGen/CGExpr.cpp
@@ -316,16 +316,13 @@
         GV->setAlignment(
             CGF.getContext().getTypeAlignInChars(M->getType()).getQuantity());
         // FIXME: Should we put the new global into a COMDAT?
-        return llvm::ConstantExpr::getBitCast(
-            GV, CGF.ConvertTypeForMem(Inner->getType())->getPointerTo());
+        return GV;
       }
     return CGF.CreateMemTemp(Inner->getType(), "ref.tmp");
 
   case SD_Thread:
   case SD_Static:
-    return llvm::ConstantExpr::getBitCast(
-        CGF.CGM.GetAddrOfGlobalTemporary(M, Inner),
-        CGF.ConvertTypeForMem(Inner->getType())->getPointerTo());
+    return CGF.CGM.GetAddrOfGlobalTemporary(M, Inner);
 
   case SD_Dynamic:
     llvm_unreachable("temporary can't have dynamic storage duration");

clang/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp

diff --git a/clang/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp b/clang/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp
index 3f92d76..382694e 100644
--- a/clang/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp
+++ b/clang/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp
@@ -482,27 +482,3 @@
     f({C(1), C(2), C(3)});
   }
 }
-
-namespace B19773010 {
-  template <class T1, class T2> struct pair {
-    T1 first;
-    T2 second;
-    constexpr pair(T1 a, T2 b) : first(a), second(b) {}
-  };
-
-  enum E { ENUM_CONSTANT };
-  struct testcase {
-    testcase(std::initializer_list<pair<const char *, E>>);
-  };
-  void f1() {
-    // CHECK-LABEL: @_ZN9B197730102f1Ev
-    testcase a{{"", ENUM_CONSTANT}};
-    // CHECK: store %"struct.B19773010::pair"* getelementptr inbounds ([1 x %"struct.B19773010::pair"], [1 x %"struct.B19773010::pair"]* bitcast ([1 x { i8*, i32 }]* @.ref.tmp{{.*}} to [1 x %"struct.B19773010::pair"]*), i64 0, i64 0), %"struct.B19773010::pair"** %{{.*}}, align 8
-  }
-  void f2() {
-    // CHECK-LABEL: @_ZN9B197730102f2Ev
-    // CHECK: store %"struct.B19773010::pair"* getelementptr inbounds ([1 x %"struct.B19773010::pair"], [1 x %"struct.B19773010::pair"]* bitcast ([1 x { i8*, i32 }]* @_ZGRZN9B197730102f2EvE1p_ to [1 x %"struct.B19773010::pair"]*), i64 0, i64 0), %"struct.B19773010::pair"** getelementptr inbounds ([2 x %"class.std::initializer_list.10"], [2 x %"class.std::initializer_list.10"]* @_ZZN9B197730102f2EvE1p, i64 0, i64 1, i32 0), align 8
-    static std::initializer_list<pair<const char *, E>> a, p[2] =
-        {a, {{"", ENUM_CONSTANT}}};
-  }
-}