[PATCH] D58295: AMDGPU: Fix memory dependence analysis by considering the offset.

Changpeng Fang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 15 11:10:41 PST 2019


cfang created this revision.
cfang added reviewers: rampitec, arsenm.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, jvesely, kzhuravl.

In SIInstrInfo::areMemAccessesTriviallyDisjoint, the current implementation ignored the offset when creates MemoryLocation
for alias analysis, and thus may report wrong result when offset is not zero. This patch fixes the issue.


https://reviews.llvm.org/D58295

Files:
  lib/Target/AMDGPU/SIInstrInfo.cpp
  test/CodeGen/AMDGPU/ds-combine-with-dependence.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58295.187050.patch
Type: text/x-patch
Size: 7065 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190215/e1684ad3/attachment.bin>


More information about the llvm-commits mailing list