[llvm] [IR][SPIR-V] Replace of PointerType::get(Type) with opaque version (NFC) (PR #124755)
Mats Jun Larsen via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 28 06:28:38 PST 2025
https://github.com/junlarsen created https://github.com/llvm/llvm-project/pull/124755
Follow-up to https://github.com/llvm/llvm-project/issues/123569
I must've missed these when I was going through uses last week.
>From a09d0289bcceb483dd239118cf3a257db372b47c Mon Sep 17 00:00:00 2001
From: Mats Jun Larsen <mats at jun.codes>
Date: Tue, 28 Jan 2025 23:27:22 +0900
Subject: [PATCH] [IR][SPIR-V] Replace of PointerType::get(Type) with opaque
version (NFC)
Follow-up to https://github.com/llvm/llvm-project/issues/123569
I must've missed these when I was going through uses last week.
---
llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp | 3 +--
llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp | 2 +-
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
index b52c793e57e968..d24fa8aafae9ac 100644
--- a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
@@ -2181,9 +2181,8 @@ getOrCreateSPIRVDeviceEventPointer(MachineIRBuilder &MIRBuilder,
OpaqueType = StructType::getTypeByName(Context, "opencl.clk_event_t");
if (!OpaqueType)
OpaqueType = StructType::create(Context, "spirv.DeviceEvent");
- unsigned SC0 = storageClassToAddressSpace(SPIRV::StorageClass::Function);
unsigned SC1 = storageClassToAddressSpace(SPIRV::StorageClass::Generic);
- Type *PtrType = PointerType::get(PointerType::get(OpaqueType, SC0), SC1);
+ Type *PtrType = PointerType::get(Context, SC1);
return GR->getOrCreateSPIRVType(PtrType, MIRBuilder);
}
diff --git a/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp b/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
index 874894ae987268..57442e38c71a30 100644
--- a/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
@@ -629,7 +629,7 @@ SPIRVGlobalRegistry::getOrCreateConstNullPtr(MachineIRBuilder &MIRBuilder,
unsigned AddressSpace = typeToAddressSpace(LLVMTy);
// Find a constant in DT or build a new one.
Constant *CP = ConstantPointerNull::get(
- PointerType::get(::getPointeeType(LLVMTy), AddressSpace));
+ PointerType::get(::getPointeeType(LLVMTy)->getContext(), AddressSpace));
Register Res = DT.find(CP, CurMF);
if (!Res.isValid()) {
LLT LLTy = LLT::pointer(AddressSpace, PointerSize);
More information about the llvm-commits
mailing list