[PATCH] D41651: AMDGPU: Add 32-bit constant address space
Marek Olšák via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 3 13:47:34 PST 2018
mareko updated this revision to Diff 128561.
mareko added a comment.
Version 3:
- Much simpler.
- Addrspacecast is never inserted.
- Unaligned loads are unaffected (work fine).
- Only scalar loads support 32-bit pointers. An address in a VGPR will fail to compile.
- https://reviews.llvm.org/D41715 (amdgpu.uniform on loads) is required for enforce scalar loads in some cases.
https://reviews.llvm.org/D41651
Files:
lib/Target/AMDGPU/AMDGPU.h
lib/Target/AMDGPU/AMDGPUCodeGenPrepare.cpp
lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp
lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp
lib/Target/AMDGPU/SIISelLowering.cpp
lib/Target/AMDGPU/SMInstructions.td
lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
test/CodeGen/AMDGPU/constant-address-space-32bit.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41651.128561.patch
Type: text/x-patch
Size: 19300 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180103/d999b70b/attachment-0001.bin>
More information about the llvm-commits
mailing list