Make VLAs usable, and make basic usage work correctly. Also, add a

simple test that actually does VLA codegen.

Note that despite the fact that the alloca isn't in the entry block, it 
should dominate all uses; this is guaranteed by the restrictions on goto 
into VLA scope in C99.

llvm-svn: 61291

1 files changed, 2 insertions, 10 deletions

diff --git a/clang/lib/CodeGen/CGDecl.cpp b/clang/lib/CodeGen/CGDecl.cpp
index 86bc3475761..db6aae6dfd7 100644
--- a/clang/lib/CodeGen/CGDecl.cpp
+++ b/clang/lib/CodeGen/CGDecl.cpp
@@ -192,17 +192,9 @@ void CodeGenFunction::EmitLocalBlockVarDecl(const VarDecl &D) {
 
     // Allocate memory for the array.
     llvm::Value *VLA = Builder.CreateAlloca(llvm::Type::Int8Ty, VLASize, "vla");
-    VLA = Builder.CreateBitCast(VLA, LElemPtrTy, "tmp");
-
-    llvm::AllocaInst *Alloc = 
-      CreateTempAlloca(LElemPtrTy, D.getIdentifier()->getName());
-    
-    // FIXME: Volatile?
-    Builder.CreateStore(VLA, Alloc);
-    
-    DeclPtr = Alloc;
+    DeclPtr = Builder.CreateBitCast(VLA, LElemPtrTy, "tmp");
   }
-  
+
   llvm::Value *&DMEntry = LocalDeclMap[&D];
   assert(DMEntry == 0 && "Decl already exists in localdeclmap!");
   DMEntry = DeclPtr;