Adjust some comments per review.
authorDale Johannesen <dalej@apple.com>
Fri, 15 Jan 2010 23:29:29 +0000 (23:29 +0000)
committerDale Johannesen <dalej@apple.com>
Fri, 15 Jan 2010 23:29:29 +0000 (23:29 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@93580 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/AsmPrinter/X86MCInstLower.cpp

index 931d66287c74e33bb1078035ecf289e31a27cc5e..d710a3f85b2091bfa3878b7d0127cc357ea131fb 100644 (file)
@@ -422,11 +422,13 @@ void X86AsmPrinter::printInstructionThroughMCStreamer(const MachineInstr *MI) {
     printLabel(MI);
     return;
   case TargetInstrInfo::DEBUG_VALUE: {
+    // FIXME: if this is implemented for another target before it goes
+    // away completely, the common part should be moved into AsmPrinter.
     if (!VerboseAsm)
       return;
     O << '\t' << MAI->getCommentString() << "DEBUG_VALUE: ";
     unsigned NOps = MI->getNumOperands();
-    // cast away const; DIetc do not take const operands for some reason
+    // cast away const; DIetc do not take const operands for some reason.
     DIVariable V((MDNode*)(MI->getOperand(NOps-1).getMetadata()));
     O << V.getName();
     O << " <- ";
@@ -435,7 +437,7 @@ void X86AsmPrinter::printInstructionThroughMCStreamer(const MachineInstr *MI) {
       assert(MI->getOperand(0).getType()==MachineOperand::MO_Register);
       printOperand(MI, 0);
     } else {
-      // Frame address.  Currently handles ESP or ESP + offset only
+      // Frame address.  Currently handles register +- offset only.
       assert(MI->getOperand(0).getType()==MachineOperand::MO_Register);
       assert(MI->getOperand(3).getType()==MachineOperand::MO_Immediate);
       O << '['; printOperand(MI, 0); O << '+'; printOperand(MI, 3); O << ']';