[llvm-branch-commits] [llvm] c15f71d - [NFC][SPIRV] Remove `SPIRVType` from comments (#181652)
via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Tue Feb 17 03:57:31 PST 2026
Author: Juan Manuel Martinez CaamaƱo
Date: 2026-02-17T12:22:23+01:00
New Revision: c15f71d279e3913ceba65e34d5834a5fc59f41a4
URL: https://github.com/llvm/llvm-project/commit/c15f71d279e3913ceba65e34d5834a5fc59f41a4
DIFF: https://github.com/llvm/llvm-project/commit/c15f71d279e3913ceba65e34d5834a5fc59f41a4.diff
LOG: [NFC][SPIRV] Remove `SPIRVType` from comments (#181652)
`SPIRVType` was deprecated and is being removed, so clean comments that
reference it.
In these contexts, `SPIRVType` used to mean _"a SPIRV type"_.
Added:
Modified:
llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
llvm/lib/Target/SPIRV/SPIRVPreLegalizer.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
index 8091bdc512421..3123a6f9ffe77 100644
--- a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
@@ -531,8 +531,9 @@ static Register buildBuiltinVariableLoad(
return LoadedRegister;
}
-/// Helper external function for assigning SPIRVType to a register, ensuring the
-/// register class and type are set in MRI. Defined in SPIRVPreLegalizer.cpp.
+/// Helper external function for assigning a SPIRV type to a register, ensuring
+/// the register class and type are set in MRI. Defined in
+/// SPIRVPreLegalizer.cpp.
extern void updateRegType(Register Reg, Type *Ty, SPIRVTypeInst SpirvTy,
SPIRVGlobalRegistry *GR, MachineIRBuilder &MIB,
MachineRegisterInfo &MRI);
@@ -3617,7 +3618,7 @@ lowerBuiltinType(const Type *OpaqueType,
// "Lower" the BuiltinType into TargetType. The following get<...>Type
// methods use the implementation details from TableGen records or
// TargetExtType parameters to either create a new OpType<...> machine
- // instruction or get an existing equivalent SPIRVType from
+ // instruction or get an existing equivalent SPIRV type from
// GlobalRegistry.
switch (TypeRecord->Opcode) {
diff --git a/llvm/lib/Target/SPIRV/SPIRVPreLegalizer.cpp b/llvm/lib/Target/SPIRV/SPIRVPreLegalizer.cpp
index 89b6975cdbdbd..3849a8b223b36 100644
--- a/llvm/lib/Target/SPIRV/SPIRVPreLegalizer.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVPreLegalizer.cpp
@@ -195,7 +195,7 @@ static void buildOpBitcast(SPIRVGlobalRegistry *GR, MachineIRBuilder &MIB,
// We lower G_BITCAST to OpBitcast here to avoid a MachineVerifier error.
// The verifier checks if the source and destination LLTs of a G_BITCAST are
//
diff erent, but this check is too strict for SPIR-V's typed pointers, which
-// may have the same LLT but
diff erent SPIRVType (e.g. pointers to
diff erent
+// may have the same LLT but
diff erent SPIRV type (e.g. pointers to
diff erent
// pointee types). By lowering to OpBitcast here, we bypass the verifier's
// check. See discussion in https://github.com/llvm/llvm-project/pull/110270
// for more context.
@@ -295,9 +295,9 @@ static void insertBitcasts(MachineFunction &MF, SPIRVGlobalRegistry *GR,
// %1 = G_ZEXT %2
// G_MEMCPY ... %2 ...
//
-// New registers have no SPIRVType and no register class info.
+// New registers have no SPIRV type and no register class info.
//
-// Set SPIRVType for GV, propagate it from GV to other instructions,
+// Set SPIRV type for GV, propagate it from GV to other instructions,
// also set register classes.
static SPIRVTypeInst propagateSPIRVType(MachineInstr *MI,
SPIRVGlobalRegistry *GR,
@@ -628,7 +628,7 @@ generateAssignInstrs(MachineFunction &MF, SPIRVGlobalRegistry *GR,
}
// Address the case when IRTranslator introduces instructions with new
- // registers without SPIRVType associated.
+ // registers without associated SPIRV type.
for (MachineBasicBlock &MBB : MF) {
for (MachineInstr &MI : MBB) {
switch (MI.getOpcode()) {
More information about the llvm-branch-commits
mailing list