[PATCH] D16723: [AMDGPU] Disassembler: Added basic disassembler for AMDGPU target.

[Sam Kolton via llvm-commits]

SamWot created this revision.
SamWot added reviewers: tstellarAMD, arsenm.
SamWot added subscribers: vpykhtin, nhaustov, llvm-commits.
Herald added a subscriber: arsenm.

Changes:
 * Added disassembler project
 * Fixed all decoding conflicts in .td files
 * Added DecoderMethod=“NONE” option to Target.td that allows to disable decoder generation for an instruction
 * Created decoding functions for VS_32 and VReg_32 register classes.
 * Added stubs for decoding all register classes.
 * Added several tests for disassembler

Disassembler only supports:
 * VI subtarget
 * VOP1 instruction encoding
 * 32-bit register operands and inline constants

[Valery]

One of the point that requires to pay attention to is how decoder conflicts were resolved:
 * Groups of target instructions were separated by using different DecoderNamespace (SICI, VI, CI) using similar to  AssemblerPredicate approach.
 * There were conflicts in IMAGE_<> instructions caused by two different reasons:
     1. dmask wasn’t specified for the output (fixed)
     2. There’re image instructions that differ only by the number of the address components but have the same encoding by the HW spec. The actual number of address components is determined by the HW at runtime using image resource descriptor starting from the VGPR encoded in an IMAGE instruction. This means that we should choose only one instruction from conflicting group to be the rule for decoder. I didn’t find the way to disable decoder generation for an arbitrary instruction and therefore made a onelinear fix to tablegen generator that would suppress decoder generation when DecoderMethod is set to “NONE”. This is a change that should be reviewed and submitted first. Otherwise I would need to specify different DecoderNamespace for every instruction in the conflicting group. I haven’t checked yet if DecoderMethod=“NONE” is not used in other targets.
     3. IMAGE_GATHER decoder generation is for now disabled and to be done later.

 [/Valery]


http://reviews.llvm.org/D16723

Files:
  lib/Target/AMDGPU/AMDGPU.td
  lib/Target/AMDGPU/AMDGPUInstructions.td
  lib/Target/AMDGPU/CIInstructions.td
  lib/Target/AMDGPU/CMakeLists.txt
  lib/Target/AMDGPU/Disassembler/AMDGPUDisassembler.cpp
  lib/Target/AMDGPU/Disassembler/AMDGPUDisassembler.h
  lib/Target/AMDGPU/Disassembler/CMakeLists.txt
  lib/Target/AMDGPU/Disassembler/LLVMBuild.txt
  lib/Target/AMDGPU/Disassembler/Makefile
  lib/Target/AMDGPU/LLVMBuild.txt
  lib/Target/AMDGPU/SIInstrFormats.td
  lib/Target/AMDGPU/SIInstrInfo.td
  lib/Target/AMDGPU/SIInstructions.td
  lib/Target/AMDGPU/VIInstructions.td
  test/MC/Disassembler/AMDGPU/lit.local.cfg
  test/MC/Disassembler/AMDGPU/mov.txt
  test/MC/Disassembler/AMDGPU/nop.txt
  utils/TableGen/FixedLenDecoderEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16723.46379.patch
Type: text/x-patch
Size: 45396 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160129/e66280a9/attachment.bin>