commit 30775778897112f79b6641e98c2722d306053f0b
author Fariborz Jahanian <fjahanian@apple.com> Wed Jun 06 22:58:50 2012 +0000
committer Fariborz Jahanian <fjahanian@apple.com> Wed Jun 06 22:58:50 2012 +0000
tree 9374226e3b6fdcc864b6cf305a42135cbfc716d5
parent 369558b6d8390899a8ae8074b363d9ecd1d66726

When doing arithmatic on vla pointer, make sure
to emit vla size to prevent an irgen crash.
// rdar://11485774


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@158117 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/CGExprScalar.cpp

diff --git a/lib/CodeGen/CGExprScalar.cpp b/lib/CodeGen/CGExprScalar.cpp
index e026554..fa7b136 100644
--- a/lib/CodeGen/CGExprScalar.cpp
+++ b/lib/CodeGen/CGExprScalar.cpp
@@ -1970,6 +1970,9 @@
   QualType elementType = pointerType->getPointeeType();
   if (const VariableArrayType *vla
         = CGF.getContext().getAsVariableArrayType(elementType)) {
+    // arithmatic on VLA pointer - make sure to emit the VLA size.
+    CGF.EmitVariablyModifiedType(elementType);
+    
     // The element count here is the total number of non-VLA elements.
     llvm::Value *numElements = CGF.getVLASize(vla).first;
 

test/CodeGen/vla.c

diff --git a/test/CodeGen/vla.c b/test/CodeGen/vla.c
index 9e62da5..d34f26b 100644
--- a/test/CodeGen/vla.c
+++ b/test/CodeGen/vla.c
@@ -142,3 +142,23 @@
   // CHECK-NEXT: ret i32 [[T7]]
   return p2 - p;
 }
+
+// rdar://11485774
+void test5(void)
+{
+  // CHECK: define void @test5(
+  int a[5], i = 0;
+  // CHECK: [[A:%.*]] = alloca [5 x i32], align 4
+  // CHECK-NEXT: [[I:%.*]] = alloca i32, align 4
+  // CHECK-NEXT: [[CL:%.*]] = alloca i32*, align 4
+  // CHECK-NEXT: store i32 0, i32* [[I]], align 4
+
+  (typeof(++i, (int (*)[i])a)){&a} += 0;
+  // CHECK-NEXT: [[Z:%.*]] = bitcast [5 x i32]* [[A]] to i32*
+  // CHECK-NEXT: store i32* [[Z]], i32** [[CL]]
+  // CHECK-NEXT: [[O:%.*]]  = load i32** [[CL]]
+  // CHECK-NEXT: [[T:%.*]] = load i32* [[I]], align 4
+  // CHECK-NEXT: [[VLAIX:%.*]] = mul nsw i32 0, [[T]]
+  // CHECK-NEXT: [[ADDPTR:%.*]] = getelementptr inbounds i32* [[O]], i32 [[VLAIX]]
+  // CHECK-NEXT: store i32* [[ADDPTR]], i32** [[CL]]
+}