From cabd96cbaa588c483908b0d3d449863764d59d20 Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Fri, 14 Sep 2012 10:30:42 +0000 Subject: Be a bit more aggressive in bailing out of this routine. Spotted by inspection by Duncan during review. My suspicion is that we would still have returned 0 anyways in this case, but doing it sooner is better. llvm-svn: 163895 --- llvm/lib/Transforms/Scalar/SROA.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'llvm/lib/Transforms/Scalar/SROA.cpp') diff --git a/llvm/lib/Transforms/Scalar/SROA.cpp b/llvm/lib/Transforms/Scalar/SROA.cpp index 97c28d442ad..27808ea8c8b 100644 --- a/llvm/lib/Transforms/Scalar/SROA.cpp +++ b/llvm/lib/Transforms/Scalar/SROA.cpp @@ -1336,7 +1336,7 @@ static Value *getNaturalGEPRecursively(IRBuilder<> &IRB, const TargetData &TD, const StructLayout *SL = TD.getStructLayout(STy); uint64_t StructOffset = Offset.getZExtValue(); - if (StructOffset > SL->getSizeInBytes()) + if (StructOffset >= SL->getSizeInBytes()) return 0; unsigned Index = SL->getElementContainingOffset(StructOffset); Offset -= APInt(Offset.getBitWidth(), SL->getElementOffset(Index)); -- cgit v1.2.3