[PATCH] D35361: CodeGen: Fix address space of MachineMemOperand
Yaxun Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 13 09:26:36 PDT 2017
yaxunl created this revision.
Herald added subscribers: Anastasia, nhaehnle, wdng.
Currently MachineMemOperand relies on MachinePointerInfo to represent its
address space. MachinePointerInfo assumes address space of Stack and
FixedStack kind is 0, which is not true for certain targets (e.g. amdgcn).
This patch let MachinePointerInfo assumes alloca address space of data layout
for Stack and FixedStack kind, therefore MachineMemOperand returns correct
address space for temporary SDNode generated during LLVM codegen.
This fixes failures of OpenCL conformance test basic/vload_private on target amdgcn---amdgizcl.
https://reviews.llvm.org/D35361
Files:
include/llvm/CodeGen/MachineMemOperand.h
include/llvm/CodeGen/PseudoSourceValue.h
lib/CodeGen/MachineInstr.cpp
lib/CodeGen/SelectionDAG/TargetLowering.cpp
test/CodeGen/AMDGPU/load-private-double16-amdgiz.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35361.106439.patch
Type: text/x-patch
Size: 7296 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170713/f9b97f78/attachment.bin>
More information about the llvm-commits
mailing list