[llvm] ce89eee - PPCInstrInfo: Fix readability-inconsistent-declaration-parameter-name clang-tidy warning

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 13 13:11:01 PDT 2020


Author: David Blaikie
Date: 2020-09-13T13:08:17-07:00
New Revision: ce89eeee16dd1e7ca6eead3b9d7f256ca583f6e1

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

LOG: PPCInstrInfo: Fix readability-inconsistent-declaration-parameter-name clang-tidy warning

Reduces the chance of confusion when calling the function with
autocomplete (will show the more accurate/informative variable name),
etc.

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/PowerPC/PPCInstrInfo.h b/llvm/lib/Target/PowerPC/PPCInstrInfo.h
index 2f867b16aa24..77ee236020a8 100644
--- a/llvm/lib/Target/PowerPC/PPCInstrInfo.h
+++ b/llvm/lib/Target/PowerPC/PPCInstrInfo.h
@@ -497,8 +497,9 @@ class PPCInstrInfo : public PPCGenInstrInfo {
   /// Get the base operand and byte offset of an instruction that reads/writes
   /// memory.
   bool getMemOperandsWithOffsetWidth(
-      const MachineInstr &MI, SmallVectorImpl<const MachineOperand *> &BaseOps,
-      int64_t &Offset, bool &OffsetIsScalable, unsigned &Width,
+      const MachineInstr &LdSt,
+      SmallVectorImpl<const MachineOperand *> &BaseOps, int64_t &Offset,
+      bool &OffsetIsScalable, unsigned &Width,
       const TargetRegisterInfo *TRI) const override;
 
   /// Returns true if the two given memory operations should be scheduled


        


More information about the llvm-commits mailing list