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 | ad41dcfd875e8d3421a5f0970b6a886b6e10b3b7 (patch) | |
tree | d69be955fa7d1a9e98b58b9b4baadf3bcf561d59 /lib/Transforms | |
parent | 8ed1ed8dfd2cf9215446c637a416a5a31f96051e (diff) | |
download | external_llvm-ad41dcfd875e8d3421a5f0970b6a886b6e10b3b7.tar.gz external_llvm-ad41dcfd875e8d3421a5f0970b6a886b6e10b3b7.tar.bz2 external_llvm-ad41dcfd875e8d3421a5f0970b6a886b6e10b3b7.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.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@163895 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
-rw-r--r-- | lib/Transforms/Scalar/SROA.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/SROA.cpp b/lib/Transforms/Scalar/SROA.cpp index 97c28d442a..27808ea8c8 100644 --- a/lib/Transforms/Scalar/SROA.cpp +++ b/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)); |