See bug 35998: https://bugs.llvm.org/show_bug.cgi?id=35998
It is not clear if atomics should support dmask=0. Currently only 0x1, 0x3 and 0xf values are allowed. Spec states that "all other values of dmask are illegal".
Depends on D42186
Paths
| Differential D42469
[AMDGPU][MC] Added support of 64-bit image atomics ClosedPublic Authored by dp on Jan 24 2018, 4:39 AM.
Details Summary See bug 35998: https://bugs.llvm.org/show_bug.cgi?id=35998 It is not clear if atomics should support dmask=0. Currently only 0x1, 0x3 and 0xf values are allowed. Spec states that "all other values of dmask are illegal". Depends on D42186
Diff Detail
Event TimelineHerald added subscribers: t-tye, tpr, dstuttard and 4 others. · View Herald TranscriptJan 24 2018, 4:39 AM artem.tamazov added inline comments.
This revision is now accepted and ready to land.Jan 25 2018, 7:17 AM
Closed by commit rL323534: [AMDGPU][MC] Added support of 64-bit image atomics (authored by dpreobra). · Explain WhyJan 26 2018, 7:45 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 131588 llvm/trunk/lib/Target/AMDGPU/Disassembler/AMDGPUDisassembler.cpp
llvm/trunk/lib/Target/AMDGPU/MIMGInstructions.td
llvm/trunk/lib/Target/AMDGPU/SIInstrInfo.td
llvm/trunk/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.h
llvm/trunk/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
llvm/trunk/test/MC/AMDGPU/mimg.s
llvm/trunk/test/MC/Disassembler/AMDGPU/mimg_vi.txt
|