[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