Skip to content

Commit daf8c14

Browse files
committed
Fix nits
1 parent c211568 commit daf8c14

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

llvm/lib/Target/AMDGPU/AMDGPULowerKernelArguments.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ static bool lowerKernelArguments(Function &F, const TargetMachine &TM) {
120120
CallInst *KernArgSegment =
121121
Builder.CreateIntrinsic(Intrinsic::amdgcn_kernarg_segment_ptr, {}, {},
122122
nullptr, F.getName() + ".kernarg.segment");
123+
KernArgSegment->addRetAttr(Attribute::NonNull);
123124
KernArgSegment->addRetAttr(
124125
Attribute::getWithDereferenceableBytes(Ctx, TotalKernArgSize));
125126

@@ -277,7 +278,6 @@ static bool lowerKernelArguments(Function &F, const TargetMachine &TM) {
277278
}
278279
}
279280

280-
KernArgSegment->addRetAttr(Attribute::NonNull);
281281
KernArgSegment->addRetAttr(
282282
Attribute::getWithAlignment(Ctx, std::max(KernArgBaseAlign, MaxAlign)));
283283

llvm/test/CodeGen/AMDGPU/lower-kernargs.ll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1121,17 +1121,17 @@ define amdgpu_kernel void @kern_noundef_global_ptr(ptr addrspace(1) noundef %ptr
11211121
; HSA-NEXT: [[KERN_NOUNDEF_GLOBAL_PTR_KERNARG_SEGMENT:%.*]] = call nonnull align 16 dereferenceable(264) ptr addrspace(4) @llvm.amdgcn.kernarg.segment.ptr()
11221122
; HSA-NEXT: [[PTR_KERNARG_OFFSET:%.*]] = getelementptr inbounds i8, ptr addrspace(4) [[KERN_NOUNDEF_GLOBAL_PTR_KERNARG_SEGMENT]], i64 0
11231123
; HSA-NEXT: [[PTR_LOAD:%.*]] = load ptr addrspace(1), ptr addrspace(4) [[PTR_KERNARG_OFFSET]], align 16, !invariant.load [[META1]]
1124-
; HSA-NEXT: store volatile ptr addrspace(1) [[PTR_LOAD]], ptr addrspace(1) undef, align 8
1124+
; HSA-NEXT: store volatile ptr addrspace(1) [[PTR_LOAD]], ptr addrspace(1) null, align 8
11251125
; HSA-NEXT: ret void
11261126
;
11271127
; MESA-LABEL: @kern_noundef_global_ptr(
11281128
; MESA-NEXT: [[KERN_NOUNDEF_GLOBAL_PTR_KERNARG_SEGMENT:%.*]] = call nonnull align 16 dereferenceable(264) ptr addrspace(4) @llvm.amdgcn.kernarg.segment.ptr()
11291129
; MESA-NEXT: [[PTR_KERNARG_OFFSET:%.*]] = getelementptr inbounds i8, ptr addrspace(4) [[KERN_NOUNDEF_GLOBAL_PTR_KERNARG_SEGMENT]], i64 36
11301130
; MESA-NEXT: [[PTR_LOAD:%.*]] = load ptr addrspace(1), ptr addrspace(4) [[PTR_KERNARG_OFFSET]], align 4, !invariant.load [[META1]]
1131-
; MESA-NEXT: store volatile ptr addrspace(1) [[PTR_LOAD]], ptr addrspace(1) undef, align 8
1131+
; MESA-NEXT: store volatile ptr addrspace(1) [[PTR_LOAD]], ptr addrspace(1) null, align 8
11321132
; MESA-NEXT: ret void
11331133
;
1134-
store volatile ptr addrspace(1) %ptr, ptr addrspace(1) undef
1134+
store volatile ptr addrspace(1) %ptr, ptr addrspace(1) null
11351135
ret void
11361136
}
11371137

0 commit comments

Comments
 (0)