Skip to content

Commit df13862

Browse files
committed
AMDGPU: Remove unnecessary pointer bitcast
1 parent 2e9b331 commit df13862

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

llvm/lib/Target/AMDGPU/AMDGPUPromoteAlloca.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1033,16 +1033,14 @@ AMDGPUPromoteAllocaImpl::getLocalSizeYZ(IRBuilder<> &Builder) {
10331033
DispatchPtr->addDereferenceableRetAttr(64);
10341034

10351035
Type *I32Ty = Type::getInt32Ty(Mod->getContext());
1036-
Value *CastDispatchPtr = Builder.CreateBitCast(
1037-
DispatchPtr, PointerType::get(I32Ty, AMDGPUAS::CONSTANT_ADDRESS));
10381036

10391037
// We could do a single 64-bit load here, but it's likely that the basic
10401038
// 32-bit and extract sequence is already present, and it is probably easier
10411039
// to CSE this. The loads should be mergeable later anyway.
1042-
Value *GEPXY = Builder.CreateConstInBoundsGEP1_64(I32Ty, CastDispatchPtr, 1);
1040+
Value *GEPXY = Builder.CreateConstInBoundsGEP1_64(I32Ty, DispatchPtr, 1);
10431041
LoadInst *LoadXY = Builder.CreateAlignedLoad(I32Ty, GEPXY, Align(4));
10441042

1045-
Value *GEPZU = Builder.CreateConstInBoundsGEP1_64(I32Ty, CastDispatchPtr, 2);
1043+
Value *GEPZU = Builder.CreateConstInBoundsGEP1_64(I32Ty, DispatchPtr, 2);
10461044
LoadInst *LoadZU = Builder.CreateAlignedLoad(I32Ty, GEPZU, Align(4));
10471045

10481046
MDNode *MD = MDNode::get(Mod->getContext(), std::nullopt);

0 commit comments

Comments
 (0)