[PATCH] D58802: [AMDGPU] Mark ds instructions as meybeAtomic

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 28 16:38:59 PST 2019


rampitec created this revision.
rampitec added reviewers: b-sumner, t-tye, kzhuravl.
Herald added subscribers: Petar.Avramovic, jfb, tpr, dstuttard, yaxunl, nhaehnle, wdng, jvesely, arsenm.

These were not recognized as potential atomics by memory legalizer.
The test was working not because legalizer did a right thing, but
because it has skipped all these instructions. When I have fixed
DS desciption test started to fail because region address has
changed from 4 to 2 a while ago.


https://reviews.llvm.org/D58802

Files:
  lib/Target/AMDGPU/DSInstructions.td
  test/CodeGen/AMDGPU/memory-legalizer-region.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58802.188811.patch
Type: text/x-patch
Size: 35479 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190301/847bb236/attachment-0001.bin>


More information about the llvm-commits mailing list