[PATCH] D120265: AMDGPU: Use the implicit kernargs for code object version 5
Changpeng Fang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 16 16:16:01 PDT 2022
cfang updated this revision to Diff 416017.
cfang marked 3 inline comments as done.
cfang added a comment.
Update based on Matt's comments:
1. Use buildPtrAdd
2. Remove a space
3. Add suffix for the enum definition and also wrap with a namespace
4. Remove the redundant def of ST (SubTarget)
5. Updated according to clang-format
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120265/new/
https://reviews.llvm.org/D120265
Files:
clang/lib/CodeGen/CGBuiltin.cpp
clang/test/CodeGenCUDA/amdgpu-workgroup-size.cu
llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
llvm/lib/Target/AMDGPU/SIDefines.h
llvm/lib/Target/AMDGPU/SIISelLowering.cpp
llvm/lib/Target/AMDGPU/SIISelLowering.h
llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.h
llvm/test/CodeGen/AMDGPU/GlobalISel/implicit-kernarg-backend-usage-global-isel.ll
llvm/test/CodeGen/AMDGPU/implicit-kernarg-backend-usage.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120265.416017.patch
Type: text/x-patch
Size: 61883 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220316/541ffde6/attachment.bin>
More information about the llvm-commits
mailing list