It turned out that DS_GWS opcodes must use even aligned registers.
A separate change is needed to address the issue in codegen.
Paths
| Differential D103185
[AMDGPU][MC][GFX90A] Corrected DS_GWS opcodes ClosedPublic Authored by dp on May 26 2021, 10:08 AM.
Details
Summary It turned out that DS_GWS opcodes must use even aligned registers. A separate change is needed to address the issue in codegen.
Diff Detail
Event TimelineHerald added subscribers: foad, kerbowa, hiraditya and 8 others. · View Herald TranscriptMay 26 2021, 10:08 AM This revision is now accepted and ready to land.May 26 2021, 10:12 AM This revision was landed with ongoing or failed builds.May 26 2021, 11:32 AM Closed by commit rG13c6568c6e20: [AMDGPU][MC][GFX90A] Corrected DS_GWS opcodes (authored by dp). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 348035 llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
llvm/test/MC/AMDGPU/gfx90a_err.s
llvm/test/MC/AMDGPU/gfx90a_err_pos.s
llvm/test/MC/AMDGPU/gfx90a_ldst_acc.s
llvm/test/MC/Disassembler/AMDGPU/gfx90a_ldst_acc.txt
|
Typo "VGRP". I guess it was copied from elsewhere in this file.