[PATCH] D127198: [AMDGPU][MC][GFX1013] Enable image_msaa_load

Wed Jun 8 03:22:49 PDT 2022

dp marked an inline comment as done.
dp added inline comments.

================
Comment at: llvm/test/MC/AMDGPU/gfx1011_supported.s:1
+// RUN: llvm-mc -arch=amdgcn -mcpu=gfx1011 -mattr=+wavefrontsize32,-wavefrontsize64 -show-encoding %s | FileCheck %s
+// RUN: llvm-mc -arch=amdgcn -mcpu=gfx1012 -mattr=+wavefrontsize32,-wavefrontsize64 -show-encoding %s | FileCheck %s
----------------
kosarev wrote:
> Will it harm if we drop the `_supported` bit from the file name? +Same below.
These tests are a bit special and I added the suffix to distinguish them from regular tests. We also have tests with `unsupported` suffix which serve the same purpose. I'm planning to add `supported` tests for other GFX9 and GFX10 subtargets.

Another option I see is to move all `supported` tests to a separate folder (e.g. `AMDGPU/subtargets`), in this case the `unsupported` suffix will be unnecessary. 

Any other ideas?

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

https://reviews.llvm.org/D127198