Implement i/n/s constraints correctly. This fixes

test/CodeGen/PowerPC/2007-02-16-InlineAsmNConstraint.ll


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@34368 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2007-02-17 06:00:35 +00:00
parent 419116fb72
commit 9ff6ee85fe

View File

@ -1856,12 +1856,26 @@ SDOperand TargetLowering::isOperandValidForConstraint(SDOperand Op,
char ConstraintLetter,
SelectionDAG &DAG) {
switch (ConstraintLetter) {
default: return SDOperand(0,0);
default: break;
case 'i': // Simple Integer or Relocatable Constant
case 'n': // Simple Integer
case 's': // Relocatable Constant
return Op; // FIXME: not right.
// These are okay if the operand is either a global variable address or a
// simple immediate value. If we have one of these, map to the TargetXXX
// version so that the value itself doesn't get selected.
if (ConstantSDNode *C = dyn_cast<ConstantSDNode>(Op)) {
// Simple constants are not allowed for 's'.
if (ConstraintLetter != 's')
return DAG.getTargetConstant(C->getValue(), Op.getValueType());
}
if (GlobalAddressSDNode *GA = dyn_cast<GlobalAddressSDNode>(Op)) {
if (ConstraintLetter != 'n')
return DAG.getTargetGlobalAddress(GA->getGlobal(), Op.getValueType(),
GA->getOffset());
}
break;
}
return SDOperand(0,0);
}
std::vector<unsigned> TargetLowering::