6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-09-27 00:21:03 +00:00
Code Issues Projects Releases Wiki Activity

Simplify ConstantExpr::getInBoundsGetElementPtr and fix a possible crash, if

Browse Source 
constant folding eliminated the GEP instruction.
 - clang was hitting this on its test suite (for x86_64, at least).


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@78698 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Daniel Dunbar
2009-08-11 18:28:09 +00:00
parent 3603d7a352
commit 3524bd7551
1 changed files with 5 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
lib/VMCore/Constants.cpp
View File

@@ -1442,14 +1442,11 @@ Constant *ConstantExpr::getGetElementPtr(Constant *C, Value* const *Idxs,
Constant *ConstantExpr::getInBoundsGetElementPtr(Constant *C, Constant *ConstantExpr::getInBoundsGetElementPtr(Constant *C,
Value* const *Idxs, Value* const *Idxs,
unsigned NumIdx) { unsigned NumIdx) {
// Get the result type of the getelementptr! Constant *Result = getGetElementPtr(C, Idxs, NumIdx);
const Type *Ty = // Set in bounds attribute, assuming constant folding didn't eliminate the
GetElementPtrInst::getIndexedType(C->getType(), Idxs, Idxs+NumIdx); // GEP.
assert(Ty && "GEP indices invalid!"); if (GEPOperator *GEP = dyn_cast<GEPOperator>(Result))
unsigned As = cast<PointerType>(C->getType())->getAddressSpace(); GEP->setIsInBounds(true);
Constant *Result = getGetElementPtrTy(PointerType::get(Ty, As), C,
Idxs, NumIdx);
cast<GEPOperator>(Result)->setIsInBounds(true);
return Result; return Result;
} }