[llvm] 81b2d1d - [NFC][PowerPC] Fix some typos in MachineCombiner comments

Jinsong Ji via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 24 13:43:09 PDT 2020


Author: Jinsong Ji
Date: 2020-06-24T20:40:57Z
New Revision: 81b2d1d112035d34fcbad1624f29bc214e67f1e5

URL: https://github.com/llvm/llvm-project/commit/81b2d1d112035d34fcbad1624f29bc214e67f1e5
DIFF: https://github.com/llvm/llvm-project/commit/81b2d1d112035d34fcbad1624f29bc214e67f1e5.diff

LOG: [NFC][PowerPC] Fix some typos in MachineCombiner comments

Added: 
    

Modified: 
    llvm/lib/Target/PowerPC/PPCInstrInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp b/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
index 7686440087aa..c65807691b01 100644
--- a/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
+++ b/llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
@@ -289,12 +289,11 @@ bool PPCInstrInfo::isAssociativeAndCommutative(const MachineInstr &Inst) const {
 // Index 0(InfoArrayIdxFMAInst): FMA instruction;
 // Index 1(InfoArrayIdxFAddInst): ADD instruction assoaicted with FMA;
 // Index 2(InfoArrayIdxFMULInst): MUL instruction assoaicted with FMA;
-// Index 3(InfoArrayIdxAddOpIdx): ADD operand index in the FMA operand list;
-// Index 4(InfoArrayIdxMULOpIdx): first MUL operand index in the FMA operand
-//                                list;
+// Index 3(InfoArrayIdxAddOpIdx): ADD operand index in FMA operands;
+// Index 4(InfoArrayIdxMULOpIdx): first MUL operand index in FMA operands;
 //                                second MUL operand index is plus 1.
 static const uint16_t FMAOpIdxInfo[][5] = {
-    // FIXME: add more FMA instructions like XSNMADDADP and so on.
+    // FIXME: Add more FMA instructions like XSNMADDADP and so on.
     {PPC::XSMADDADP, PPC::XSADDDP, PPC::XSMULDP, 1, 2},
     {PPC::XSMADDASP, PPC::XSADDSP, PPC::XSMULSP, 1, 2},
     {PPC::XVMADDADP, PPC::XVADDDP, PPC::XVMULDP, 1, 2},
@@ -362,12 +361,12 @@ bool PPCInstrInfo::getFMAPatterns(
       return false;
 
     // Instruction can be reassociated.
-    // fast match flags may prohibit reassociation.
+    // fast math flags may prohibit reassociation.
     if (!(Instr.getFlag(MachineInstr::MIFlag::FmReassoc) &&
           Instr.getFlag(MachineInstr::MIFlag::FmNsz)))
       return false;
 
-    // Instruction operands are virtual registers for reassociating.
+    // Instruction operands are virtual registers for reassociation.
     if (!IsAllOpsVirtualReg(Instr))
       return false;
 
@@ -544,7 +543,7 @@ void PPCInstrInfo::reassociateFMA(
             .addReg(RegY, getKillRegState(KillY))
             .addReg(RegM31, getKillRegState(KillM31))
             .addReg(RegM32, getKillRegState(KillM32));
-    // if AddOpIdx is not 1, adjust the order.
+    // If AddOpIdx is not 1, adjust the order.
     if (AddOpIdx != 1) {
       AdjustOperandOrder(MINewB, RegX, KillX, RegM21, KillM21, RegM22, KillM22);
       AdjustOperandOrder(MINewA, RegY, KillY, RegM31, KillM31, RegM32, KillM32);
@@ -556,7 +555,7 @@ void PPCInstrInfo::reassociateFMA(
             .addReg(NewVRB, getKillRegState(true))
             .addReg(NewVRA, getKillRegState(true));
 
-    // update flags for new created instructions.
+    // Update flags for newly created instructions.
     setSpecialOperandAttr(*MINewA, IntersectedFlags);
     setSpecialOperandAttr(*MINewB, IntersectedFlags);
     setSpecialOperandAttr(*MINewC, IntersectedFlags);
@@ -596,7 +595,7 @@ void PPCInstrInfo::reassociateFMA(
             .addReg(NewVRB, getKillRegState(true))
             .addReg(NewVRD, getKillRegState(true));
 
-    // update flags for new created instructions.
+    // Update flags for newly created instructions.
     setSpecialOperandAttr(*MINewA, IntersectedFlags);
     setSpecialOperandAttr(*MINewB, IntersectedFlags);
     setSpecialOperandAttr(*MINewD, IntersectedFlags);


        


More information about the llvm-commits mailing list