[PATCH] D158465: [AMDGPU] Rename 64BitDPP feature and fix the checks

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 22 10:30:23 PDT 2023


rampitec added inline comments.


================
Comment at: llvm/test/MC/Disassembler/AMDGPU/gfx940_features.txt:481
-# GFX940: v_cvt_pk_f32_fp8_dpp v[0:1], v3 quad_perm:[0,2,1,1] row_mask:0xf bank_mask:0xf ; encoding: [0xfa,0xac,0x00,0x7e,0x03,0x58,0x00,0xff]
-0xfa,0xac,0x00,0x7e,0x03,0x58,0x00,0xff
 
----------------
Joe_Nash wrote:
> Add a disassembler error test to ensure this is forbidden?
Disasm is more permissive, it does not check this, just as many other errors.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158465/new/

https://reviews.llvm.org/D158465



More information about the llvm-commits mailing list