[PATCH] D109616: [AMDGPU][MC][GFX7][GFX10] Corrected image_atomic_fcmpswap

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 13 03:37:24 PDT 2021


critson added inline comments.


================
Comment at: llvm/test/MC/Disassembler/AMDGPU/gfx10_mimg.txt:255-266
-# GFX10: image_sample_c_b v[16:19], v[8:12], s[20:27], s[100:103] dmask:0xf dim:SQ_RSRC_IMG_CUBE ; encoding: [0x18,0x0f,0xb4,0xf0,0x08,0x10,0x25,0x03]
-0x18,0x0f,0xb4,0xf0,0x08,0x10,0x25,0x03
-
-# GFX10: image_sample_c_b_cl v[16:19], v[250:255], s[20:27], s[100:103] dmask:0xf dim:SQ_RSRC_IMG_CUBE ; encoding: [0x18,0x0f,0xb8,0xf0,0xfa,0x10,0x25,0x03]
-0x18,0x0f,0xb8,0xf0,0xfa,0x10,0x25,0x03
-
-# GFX10: image_sample_c_lz v[16:19], v[253:255], s[20:27], s[100:103] dmask:0xf dim:SQ_RSRC_IMG_2D ; encoding: [0x08,0x0f,0xbc,0xf0,0xfd,0x10,0x25,0x03]
----------------
dp wrote:
> critson wrote:
> > How do these changes relate to this patch?
> I attempted to bring some organization to the tests so I moved these to non-NSA section.
Thanks for the clarification. Reorganisation makes sense.


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

https://reviews.llvm.org/D109616



More information about the llvm-commits mailing list