The way we are trying to figure out banksel immediate operand may yield different results for different type of insns. This will eventually need to be changed but currently let us prevent the crash in cases of incorrect detection of banksel operand.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@68713 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Sanjiv Gupta 2009-04-09 17:06:24 +00:00
parent 0c8382ce9a
commit 37831d0a12

View File

@ -77,8 +77,9 @@ bool PIC16AsmPrinter::printMachineInstruction(const MachineInstr *MI) {
// This is not true for external variables as section names for global // This is not true for external variables as section names for global
// variables in all files are same at this time. For eg. initialized // variables in all files are same at this time. For eg. initialized
// data in put in idata.# section in all files. // data in put in idata.# section in all files.
if (((int)BS.getImm() == 1) && if ((BS.getType() == MachineOperand::MO_Immediate
((Op.isGlobal() && Op.getGlobal()->hasExternalLinkage()) || && (int)BS.getImm() == 1)
&& ((Op.isGlobal() && Op.getGlobal()->hasExternalLinkage()) ||
(NewBank.compare(CurBank) != 0))) { (NewBank.compare(CurBank) != 0))) {
O << "\tbanksel "; O << "\tbanksel ";
printOperand(MI, BankSelVar); printOperand(MI, BankSelVar);