Reuse the values if they are constants: this is important so that we index into the right structure field

This fixes bug: BasicAA/2003-07-03-BasicAACrash.ll

llvm-svn: 7093
This commit is contained in:
Chris Lattner 2003-07-03 06:42:38 +00:00
parent 6642d2a0c8
commit b93ef28cef

View File

@ -288,8 +288,13 @@ BasicAliasAnalysis::CheckGEPInstructions(GetElementPtrInst *GEP1, unsigned G1S,
const Value *Op1 = GEP1->getOperand(i);
const Value *Op2 = GEP2->getOperand(i);
if (Op1 == Op2) { // If they are equal, use a zero index...
if (!isa<Constant>(Op1)) {
Indices1.push_back(Constant::getNullValue(Op1->getType()));
Indices2.push_back(Indices1.back());
} else {
Indices1.push_back((Value*)Op1);
Indices2.push_back((Value*)Op2);
}
} else {
if (const ConstantInt *Op1C = dyn_cast<ConstantInt>(Op1)) {
// If this is an array index, make sure the array element is in range...