[PATCH] D109733: [AMDGPU] NFC: Fixing small spelling errors in AMDGPU header files
Jacob Lambert via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 13 19:34:07 PDT 2021
lamb-j created this revision.
lamb-j added reviewers: kerbowa, msearles, tstellar.
Herald added subscribers: foad, hiraditya, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl, arsenm.
lamb-j requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.
Nonfunctional commit fixing several minor spelling errors in llvm/lib/Target/AMDGPU header files.
Testing workflow as a new contributor.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D109733
Files:
llvm/lib/Target/AMDGPU/AMDGPU.h
llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.h
llvm/lib/Target/AMDGPU/GCNSchedStrategy.h
llvm/lib/Target/AMDGPU/GCNSubtarget.h
llvm/lib/Target/AMDGPU/R600InstrInfo.h
Index: llvm/lib/Target/AMDGPU/R600InstrInfo.h
===================================================================
--- llvm/lib/Target/AMDGPU/R600InstrInfo.h
+++ llvm/lib/Target/AMDGPU/R600InstrInfo.h
@@ -211,7 +211,7 @@
bool expandPostRAPseudo(MachineInstr &MI) const override;
- /// Reserve the registers that may be accesed using indirect addressing.
+ /// Reserve the registers that may be accessed using indirect addressing.
void reserveIndirectRegisters(BitVector &Reserved,
const MachineFunction &MF,
const R600RegisterInfo &TRI) const;
@@ -220,7 +220,7 @@
/// \p Channel
///
/// We model indirect addressing using a virtual address space that can be
- /// accesed with loads and stores. The "Indirect Address" is the memory
+ /// accessed with loads and stores. The "Indirect Address" is the memory
/// address in this virtual address space that maps to the given \p RegIndex
/// and \p Channel.
unsigned calculateIndirectAddress(unsigned RegIndex, unsigned Channel) const;
Index: llvm/lib/Target/AMDGPU/GCNSubtarget.h
===================================================================
--- llvm/lib/Target/AMDGPU/GCNSubtarget.h
+++ llvm/lib/Target/AMDGPU/GCNSubtarget.h
@@ -265,7 +265,7 @@
return (Generation)Gen;
}
- /// Return the number of high bits known to be zero fror a frame index.
+ /// Return the number of high bits known to be zero for a frame index.
unsigned getKnownHighZeroBitsForFrameIndex() const {
return countLeadingZeros(MaxWaveScratchSize) + getWavefrontSizeLog2();
}
@@ -605,7 +605,7 @@
}
/// Return if most LDS instructions have an m0 use that require m0 to be
- /// iniitalized.
+ /// initialized.
bool ldsRequiresM0Init() const {
return getGeneration() < GFX9;
}
@@ -746,7 +746,7 @@
}
// Scratch is allocated in 256 dword per wave blocks for the entire
- // wavefront. When viewed from the perspecive of an arbitrary workitem, this
+ // wavefront. When viewed from the perspective of an arbitrary workitem, this
// is 4-byte aligned.
//
// Only 4-byte alignment is really needed to access anything. Transformations
Index: llvm/lib/Target/AMDGPU/GCNSchedStrategy.h
===================================================================
--- llvm/lib/Target/AMDGPU/GCNSchedStrategy.h
+++ llvm/lib/Target/AMDGPU/GCNSchedStrategy.h
@@ -54,7 +54,7 @@
// before a region scheduling to know if the region had such clusters.
bool HasClusteredNodes;
- // schedule() have seen a an excess register pressure and had to track
+ // schedule() have seen an excess register pressure and had to track
// register pressure for actual scheduling heuristics.
bool HasExcessPressure;
Index: llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.h
===================================================================
--- llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.h
+++ llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.h
@@ -124,7 +124,7 @@
std::unique_ptr<CSEConfigBase> getCSEConfig() const override;
/// Check if a pass is enabled given \p Opt option. The option always
- /// overrides defaults if explicitely used. Otherwise its default will
+ /// overrides defaults if explicitly used. Otherwise its default will
/// be used given that a pass shall work at an optimization \p Level
/// minimum.
bool isPassEnabled(const cl::opt<bool> &Opt,
Index: llvm/lib/Target/AMDGPU/AMDGPU.h
===================================================================
--- llvm/lib/Target/AMDGPU/AMDGPU.h
+++ llvm/lib/Target/AMDGPU/AMDGPU.h
@@ -358,9 +358,9 @@
BUFFER_FAT_POINTER = 7, ///< Address space for 160-bit buffer fat pointers.
- /// Address space for direct addressible parameter memory (CONST0).
+ /// Address space for direct addressable parameter memory (CONST0).
PARAM_D_ADDRESS = 6,
- /// Address space for indirect addressible parameter memory (VTX1).
+ /// Address space for indirect addressable parameter memory (VTX1).
PARAM_I_ADDRESS = 7,
// Do not re-order the CONSTANT_BUFFER_* enums. Several places depend on
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109733.372387.patch
Type: text/x-patch
Size: 4144 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210914/923b4c7e/attachment.bin>
More information about the llvm-commits
mailing list