This is an archive of the discontinued LLVM Phabricator instance.

Differential D126919

[wip][AsmPrinter] Add assertion for out-of-bounds access
DraftPublic

Authored by smeenai on Jun 2 2022, 1:48 PM.
This is a draft revision that has not yet been submitted for review.

Details

Reviewers
None

Diff Detail

Revision Contents

PathSize
llvm/
lib/
CodeGen/
AsmPrinter/
1 line

Diff 433875

llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp

Show First 20 Lines • Show All 1,263 Lines • ▼ Show 20 Linesvoid AsmPrinter::emitCFIInstruction(const MachineInstr &MI) {
while (I != MBB->end() && I->isTransient()) while (I != MBB->end() && I->isTransient())
++I; ++I;
if (I == MBB->instr_end() && if (I == MBB->instr_end() &&
MBB->getReverseIterator() == MBB->getParent()->rbegin()) MBB->getReverseIterator() == MBB->getParent()->rbegin())
return; return;
const std::vector<MCCFIInstruction> &Instrs = MF->getFrameInstructions(); const std::vector<MCCFIInstruction> &Instrs = MF->getFrameInstructions();
unsigned CFIIndex = MI.getOperand(0).getCFIIndex(); unsigned CFIIndex = MI.getOperand(0).getCFIIndex();
assert(CFIIndex < Instrs.size() && "CFIIndex out of range");
const MCCFIInstruction &CFI = Instrs[CFIIndex]; const MCCFIInstruction &CFI = Instrs[CFIIndex];
emitCFIInstruction(CFI); emitCFIInstruction(CFI);
} }
void AsmPrinter::emitFrameAlloc(const MachineInstr &MI) { void AsmPrinter::emitFrameAlloc(const MachineInstr &MI) {
// The operands are the MCSymbol and the frame offset of the allocation. // The operands are the MCSymbol and the frame offset of the allocation.
MCSymbol *FrameAllocSym = MI.getOperand(0).getMCSymbol(); MCSymbol *FrameAllocSym = MI.getOperand(0).getMCSymbol();
int FrameOffset = MI.getOperand(1).getImm(); int FrameOffset = MI.getOperand(1).getImm();
▲ Show 20 Lines • Show All 2,553 Lines • Show Last 20 Lines