See bug 39297: https://bugs.llvm.org/show_bug.cgi?id=39297
Note that these operands are defined as 32-bits only. However in MC assembler they should be enabled for both 32-bit and 64-bit operands.
Paths
| Differential D59290
[AMDGPU][MC][GFX9] Added support of operands shared_base, shared_limit, private_base, private_limit, pops_exiting_wave_id ClosedPublic Authored by dp on Mar 13 2019, 3:17 AM.
Details Summary See bug 39297: https://bugs.llvm.org/show_bug.cgi?id=39297 Note that these operands are defined as 32-bits only. However in MC assembler they should be enabled for both 32-bit and 64-bit operands.
Diff Detail
Event TimelineHerald added subscribers: jfb, t-tye, tpr and 6 others. · View Herald TranscriptMar 13 2019, 3:17 AM This revision is now accepted and ready to land.Mar 13 2019, 9:12 AM Comment Actions After a discussion with HW people it turned out that these operands (shared_base, shared_limit etc) differ from inline constants in that they compete with other scalars for constant bus access. This change includes the following corrections:
Closed by commit rL356561: [AMDGPU][MC][GFX9] Added support of operands shared_base, shared_limit… (authored by dpreobra). · Explain WhyMar 20 2019, 8:39 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 191501 llvm/trunk/lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp
llvm/trunk/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
llvm/trunk/lib/Target/AMDGPU/Disassembler/AMDGPUDisassembler.cpp
llvm/trunk/lib/Target/AMDGPU/InstPrinter/AMDGPUInstPrinter.cpp
llvm/trunk/lib/Target/AMDGPU/SIRegisterInfo.cpp
llvm/trunk/lib/Target/AMDGPU/SIRegisterInfo.td
llvm/trunk/test/MC/AMDGPU/literals.s
llvm/trunk/test/MC/Disassembler/AMDGPU/literal_gfx9.txt
|