[llvm-commits] [llvm] r93580 - /llvm/trunk/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp

[Dale Johannesen]

Author: johannes
Date: Fri Jan 15 17:29:29 2010
New Revision: 93580

URL: http://llvm.org/viewvc/llvm-project?rev=93580&view=rev
Log:
Adjust some comments per review.


Modified:
    llvm/trunk/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp

Modified: llvm/trunk/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp?rev=93580&r1=93579&r2=93580&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp (original)
+++ llvm/trunk/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp Fri Jan 15 17:29:29 2010
@@ -422,11 +422,13 @@
     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 @@
       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 << ']';