[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