diff options
| author | Chandler Carruth <chandlerc@gmail.com> | 2012-09-14 10:30:42 +0000 |
|---|---|---|
| committer | Chandler Carruth <chandlerc@gmail.com> | 2012-09-14 10:30:42 +0000 |
| commit | cabd96cbaa588c483908b0d3d449863764d59d20 (patch) | |
| tree | 8eef656407c81c7e12350993e827537bac654896 /llvm/lib/Transforms/Scalar/SROA.cpp | |
| parent | dd3cea898fe27110ffa948792854c47f9138b8f6 (diff) | |
| download | bcm5719-llvm-cabd96cbaa588c483908b0d3d449863764d59d20.tar.gz bcm5719-llvm-cabd96cbaa588c483908b0d3d449863764d59d20.zip | |
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
Diffstat (limited to 'llvm/lib/Transforms/Scalar/SROA.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/SROA.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
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)); |
