[llvm] r355848 - [NFC][PowerPC] Add comment for PPCAsmPrinter::printOperand

Jinsong Ji via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 11 10:57:49 PDT 2019


Author: jsji
Date: Mon Mar 11 10:57:49 2019
New Revision: 355848

URL: http://llvm.org/viewvc/llvm-project?rev=355848&view=rev
Log:
[NFC][PowerPC] Add comment for PPCAsmPrinter::printOperand

Patch by Yi-Hong Lyu

Modified:
    llvm/trunk/lib/Target/PowerPC/PPCAsmPrinter.cpp

Modified: llvm/trunk/lib/Target/PowerPC/PPCAsmPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PowerPC/PPCAsmPrinter.cpp?rev=355848&r1=355847&r2=355848&view=diff
==============================================================================
--- llvm/trunk/lib/Target/PowerPC/PPCAsmPrinter.cpp (original)
+++ llvm/trunk/lib/Target/PowerPC/PPCAsmPrinter.cpp Mon Mar 11 10:57:49 2019
@@ -96,6 +96,9 @@ public:
 
     void EmitInstruction(const MachineInstr *MI) override;
 
+    /// This function is for PrintAsmOperand and PrintAsmMemoryOperand,
+    /// invoked by EmitMSInlineAsmStr and EmitGCCInlineAsmStr only.
+    /// The \p MI would be INLINEASM ONLY.
     void printOperand(const MachineInstr *MI, unsigned OpNo, raw_ostream &O);
 
     bool PrintAsmOperand(const MachineInstr *MI, unsigned OpNo,
@@ -164,8 +167,9 @@ void PPCAsmPrinter::printOperand(const M
 
   switch (MO.getType()) {
   case MachineOperand::MO_Register: {
-    unsigned Reg = PPCInstrInfo::getRegNumForOperand(MI->getDesc(),
-                                                     MO.getReg(), OpNo);
+    // The MI is INLINEASM ONLY and UseVSXReg is always false.
+    unsigned Reg =
+        PPCInstrInfo::getRegNumForOperand(MI->getDesc(), MO.getReg(), OpNo);
 
     const char *RegName = PPCInstPrinter::getRegisterName(Reg);
 




More information about the llvm-commits mailing list