Don't put constants into the scalar map!

llvm-svn: 4657
This commit is contained in:
Chris Lattner 2002-11-09 20:14:03 +00:00
parent e742f31e0c
commit 2dd1adafa6
1 changed files with 13 additions and 9 deletions

View File

@ -150,30 +150,34 @@ DSNodeHandle GraphBuilder::getValueDest(Value &Val) {
if (V == Constant::getNullValue(V->getType()))
return 0; // Null doesn't point to anything, don't add to ScalarMap!
DSNodeHandle &NH = ScalarMap[V];
if (NH.getNode())
return NH; // Already have a node? Just return it...
if (Constant *C = dyn_cast<Constant>(V))
if (ConstantPointerRef *CPR = dyn_cast<ConstantPointerRef>(C)) {
return NH = getValueDest(*CPR->getValue());
return getValueDest(*CPR->getValue());
} else if (ConstantExpr *CE = dyn_cast<ConstantExpr>(C)) {
if (CE->getOpcode() == Instruction::Cast)
return NH = getValueDest(*CE->getOperand(0));
return getValueDest(*CE->getOperand(0));
if (CE->getOpcode() == Instruction::GetElementPtr) {
visitGetElementPtrInst(*CE);
return ScalarMap[CE];
std::map<Value*, DSNodeHandle>::iterator I = ScalarMap.find(CE);
assert(I != ScalarMap.end() && "GEP didn't get processed right?");
DSNodeHandle NH = I->second;
ScalarMap.erase(I); // Remove constant from scalarmap
return NH;
}
// This returns a conservative unknown node for any unhandled ConstExpr
return NH = createNode(DSNode::UnknownNode);
return createNode(DSNode::UnknownNode);
} else if (ConstantIntegral *CI = dyn_cast<ConstantIntegral>(C)) {
// Random constants are unknown mem
return NH = createNode(DSNode::UnknownNode);
return createNode(DSNode::UnknownNode);
} else {
assert(0 && "Unknown constant type!");
}
DSNodeHandle &NH = ScalarMap[V];
if (NH.getNode())
return NH; // Already have a node? Just return it...
// Otherwise we need to create a new node to point to...
DSNode *N;
if (GlobalValue *GV = dyn_cast<GlobalValue>(V)) {