diff options
| -rw-r--r-- | lib/VMCore/Instructions.cpp | 2 | ||||
| -rw-r--r-- | test/Verifier/2002-11-05-GetelementptrPointers.ll | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/lib/VMCore/Instructions.cpp b/lib/VMCore/Instructions.cpp index c54815a..9fc70ce 100644 --- a/lib/VMCore/Instructions.cpp +++ b/lib/VMCore/Instructions.cpp @@ -1336,7 +1336,7 @@ const Type* ExtractValueInst::getIndexedType(const Type *Agg, unsigned CurIdx = 0; for (; CurIdx != NumIdx; ++CurIdx) { const CompositeType *CT = dyn_cast<CompositeType>(Agg); - if (!CT) return 0; + if (!CT || isa<PointerType>(CT)) return 0; Value *Index = Idxs[CurIdx]; if (!CT->indexValid(Index)) return 0; Agg = CT->getTypeAtIndex(Index); diff --git a/test/Verifier/2002-11-05-GetelementptrPointers.ll b/test/Verifier/2002-11-05-GetelementptrPointers.ll index c762b2a..e37a0ff 100644 --- a/test/Verifier/2002-11-05-GetelementptrPointers.ll +++ b/test/Verifier/2002-11-05-GetelementptrPointers.ll @@ -1,7 +1,7 @@ -; RUN: llvm-as < %s +; RUN: not llvm-as < %s |& grep {Invalid getelementptr indices} -; This testcase was previously considered invalid for indexing into a pointer -; that is contained WITHIN a structure, but this is now valid. +; This testcase is invalid because we are indexing into a pointer that is +; contained WITHIN a structure. define void @test({i32, i32*} * %X) { getelementptr {i32, i32*} * %X, i32 0, i32 1, i32 0 |