diff options
author | David Blaikie <dblaikie@gmail.com> | 2013-06-15 15:52:58 +0000 |
---|---|---|
committer | David Blaikie <dblaikie@gmail.com> | 2013-06-15 15:52:58 +0000 |
commit | f14b44c71b3a58c3aef8e277dd3817cb216b5661 (patch) | |
tree | 6e1702d2e22384ef0db81d1e1cf0f3397a53ba26 /lib/CodeGen/AsmPrinter/AsmPrinter.cpp | |
parent | 515971fdd77b82fe3dabdbaf34524cafc3f08415 (diff) | |
download | external_llvm-f14b44c71b3a58c3aef8e277dd3817cb216b5661.zip external_llvm-f14b44c71b3a58c3aef8e277dd3817cb216b5661.tar.gz external_llvm-f14b44c71b3a58c3aef8e277dd3817cb216b5661.tar.bz2 |
DebugInfo: print DBG_VALUE MachineInstrs with [] for deref and drop the offset when it's zero
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@184045 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/AsmPrinter/AsmPrinter.cpp')
-rw-r--r-- | lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp index 7f4ba7a..9b39264 100644 --- a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp +++ b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp @@ -569,6 +569,8 @@ static bool emitDebugValueComment(const MachineInstr *MI, AsmPrinter &AP) { } OS << V.getName() << " <- "; + int64_t Offset = MI->getOperand(1).getImm(); + bool Deref = false; // Register or immediate value. Register 0 means undef. if (MI->getOperand(0).isFPImm()) { APFloat APF = APFloat(MI->getOperand(0).getFPImm()->getValueAPF()); @@ -590,17 +592,25 @@ static bool emitDebugValueComment(const MachineInstr *MI, AsmPrinter &AP) { MI->getOperand(0).getCImm()->getValue().print(OS, false /*isSigned*/); } else { assert(MI->getOperand(0).isReg() && "Unknown operand type"); - if (MI->getOperand(0).getReg() == 0) { + unsigned Reg = MI->getOperand(0).getReg(); + if (Reg == 0) { // Suppress offset, it is not meaningful here. OS << "undef"; // NOTE: Want this comment at start of line, don't emit with AddComment. AP.OutStreamer.EmitRawText(OS.str()); return true; } - OS << AP.TM.getRegisterInfo()->getName(MI->getOperand(0).getReg()); + Deref = Offset != 0; // FIXME: use a better sentinel value so that deref of + // a reg with a zero offset is valid + if (Deref) + OS << '['; + OS << AP.TM.getRegisterInfo()->getName(Reg); } - OS << '+' << MI->getOperand(1).getImm(); + if (Offset) + OS << '+' << Offset; + if (Deref) + OS << ']'; // NOTE: Want this comment at start of line, don't emit with AddComment. AP.OutStreamer.EmitRawText(OS.str()); return true; |