Fix PR3304
authorChris Lattner <sabre@nondot.org>
Fri, 9 Jan 2009 18:18:43 +0000 (18:18 +0000)
committerChris Lattner <sabre@nondot.org>
Fri, 9 Jan 2009 18:18:43 +0000 (18:18 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61995 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Scalar/ScalarReplAggregates.cpp
test/Transforms/ScalarRepl/2009-01-09-scalarrepl-empty.ll [new file with mode: 0644]

index 041fd4917b4b19d91e3b4e1d9d1d30133d20149e..d7b8b58ab0846ea864663276a64973f84610c9c1 100644 (file)
@@ -856,6 +856,10 @@ void SROA::RewriteStoreUserOfWholeAlloca(StoreInst *SI,
       
       // Truncate down to an integer of the right size.
       uint64_t FieldSizeBits = TD->getTypeSizeInBits(FieldTy);
+      
+      // Ignore zero sized fields like {}, they obviously contain no data.
+      if (FieldSizeBits == 0) continue;
+      
       if (FieldSizeBits != AllocaSizeBits)
         EltVal = new TruncInst(EltVal, IntegerType::get(FieldSizeBits), "", SI);
       Value *DestField = NewElts[i];
@@ -887,6 +891,8 @@ void SROA::RewriteStoreUserOfWholeAlloca(StoreInst *SI,
       Shift = 0;
     
     for (unsigned i = 0, e = NewElts.size(); i != e; ++i) {
+      // Ignore zero sized fields like {}, they obviously contain no data.
+      if (ElementSizeBits == 0) continue;
       
       Value *EltVal = SrcVal;
       if (Shift) {
@@ -959,8 +965,12 @@ void SROA::RewriteLoadUserOfWholeAlloca(LoadInst *LI, AllocationInst *AI,
     Value *SrcField = NewElts[i];
     const Type *FieldTy =
       cast<PointerType>(SrcField->getType())->getElementType();
-    const IntegerType *FieldIntTy = 
-      IntegerType::get(TD->getTypeSizeInBits(FieldTy));
+    uint64_t FieldSizeBits = TD->getTypeSizeInBits(FieldTy);
+    
+    // Ignore zero sized fields like {}, they obviously contain no data.
+    if (FieldSizeBits == 0) continue;
+    
+    const IntegerType *FieldIntTy = IntegerType::get(FieldSizeBits);
     if (!isa<IntegerType>(FieldTy) && !FieldTy->isFloatingPoint() &&
         !isa<VectorType>(FieldTy))
       SrcField = new BitCastInst(SrcField, PointerType::getUnqual(FieldIntTy),
diff --git a/test/Transforms/ScalarRepl/2009-01-09-scalarrepl-empty.ll b/test/Transforms/ScalarRepl/2009-01-09-scalarrepl-empty.ll
new file mode 100644 (file)
index 0000000..9cdf4a0
--- /dev/null
@@ -0,0 +1,15 @@
+; RUN: llvm-as < %s | opt -scalarrepl | llvm-dis
+; PR3304
+
+       %struct.c37304a__vrec = type { i8, %struct.c37304a__vrec___disc___XVN }
+        %struct.c37304a__vrec___disc___XVN = type {
+%struct.c37304a__vrec___disc___XVN___O }
+        %struct.c37304a__vrec___disc___XVN___O = type {  }
+
+define void @_ada_c37304a() {
+entry:
+        %v = alloca %struct.c37304a__vrec               ;
+        %0 = getelementptr %struct.c37304a__vrec* %v, i32 0, i32 0             
+        store i8 8, i8* %0, align 1
+        unreachable
+}