summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChris Lattner <sabre@nondot.org>2010-04-17 19:02:33 +0000
committerChris Lattner <sabre@nondot.org>2010-04-17 19:02:33 +0000
commit0a8d91a816b72899bc1038be1131045ff62016ec (patch)
tree35bf6a3a897ac140d2d8eaf2cbeaf7c2d9af510a
parentb927073f2ef00bfa88656dd228e700be52599b7d (diff)
downloadbcm5719-llvm-0a8d91a816b72899bc1038be1131045ff62016ec.tar.gz
bcm5719-llvm-0a8d91a816b72899bc1038be1131045ff62016ec.zip
fix PR6332, allowing an index of zero into a zero sized array
even if the element of the array has no size. llvm-svn: 101662
-rw-r--r--llvm/lib/Target/TargetData.cpp4
-rw-r--r--llvm/test/CodeGen/Generic/crash.ll8
2 files changed, 10 insertions, 2 deletions
diff --git a/llvm/lib/Target/TargetData.cpp b/llvm/lib/Target/TargetData.cpp
index abee2e19244..d89fafd8137 100644
--- a/llvm/lib/Target/TargetData.cpp
+++ b/llvm/lib/Target/TargetData.cpp
@@ -630,8 +630,8 @@ uint64_t TargetData::getIndexedOffset(const Type *ptrTy, Value* const* Indices,
Ty = cast<SequentialType>(Ty)->getElementType();
// Get the array index and the size of each array element.
- int64_t arrayIdx = cast<ConstantInt>(Indices[CurIDX])->getSExtValue();
- Result += arrayIdx * (int64_t)getTypeAllocSize(Ty);
+ if (int64_t arrayIdx = cast<ConstantInt>(Indices[CurIDX])->getSExtValue())
+ Result += arrayIdx * (int64_t)getTypeAllocSize(Ty);
}
}
diff --git a/llvm/test/CodeGen/Generic/crash.ll b/llvm/test/CodeGen/Generic/crash.ll
new file mode 100644
index 00000000000..7218565617f
--- /dev/null
+++ b/llvm/test/CodeGen/Generic/crash.ll
@@ -0,0 +1,8 @@
+; RUN: llc %s -o -
+
+; PR6332
+%struct.AVCodecTag = type opaque
+@ff_codec_bmp_tags = external global [0 x %struct.AVCodecTag]
+@tags = global [1 x %struct.AVCodecTag*] [%struct.AVCodecTag* getelementptr
+inbounds ([0 x %struct.AVCodecTag]* @ff_codec_bmp_tags, i32 0, i32 0)]
+
OpenPOWER on IntegriCloud