[llvm] [AMDGPU] Remove some Wave64 checks for GFX10+ MC tests (NFC) (PR #100971)

Changpeng Fang via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 31 23:35:10 PDT 2024


================
@@ -1,5 +1,4 @@
-// RUN: llvm-mc -triple=amdgcn -mcpu=gfx1200 -mattr=+wavefrontsize32 -show-encoding %s | FileCheck --check-prefixes=GFX12 %s
-// RUN: llvm-mc -triple=amdgcn -mcpu=gfx1200 -mattr=+wavefrontsize64 -show-encoding %s | FileCheck --check-prefixes=GFX12 %s
----------------
changpeng wrote:

Okay. you are thinking every instruction should be verified on every possible wavefrontsize.
This patch thinks if we are 100% sure same instruction will be generated on both wave sizes, we only need to verify for one wave size.  Both are reasonable and will not lose anything.
I am going to abandon this patch if there is no other opinions. Thanks. 

https://github.com/llvm/llvm-project/pull/100971


More information about the llvm-commits mailing list