[llvm] [AMDGPU][True16][test] update VOPC/VOPCX test for true16/fake16 (PR #112829)

Ivan Kosarev via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 05:04:43 PDT 2024


================
@@ -0,0 +1,10487 @@
+; NOTE: Assertions have been autogenerated by utils/update_mc_test_checks.py UTC_ARGS: --sort --version 5
+// RUN: not llvm-mc -triple=amdgcn -mcpu=gfx1100 -mattr=+wavefrontsize32,-real-true16 -show-encoding %s | FileCheck --check-prefixes=GFX11,W32 %s
+// RUN: not llvm-mc -triple=amdgcn -mcpu=gfx1100 -mattr=+wavefrontsize64,-real-true16 -show-encoding %s | FileCheck --check-prefixes=GFX11,W64 %s
+// RUN: not llvm-mc -triple=amdgcn -mcpu=gfx1100 -mattr=+wavefrontsize32,-real-true16 %s 2>&1 > /dev/null | FileCheck --check-prefix=W32-ERR --implicit-check-not=error: %s
+// RUN: not llvm-mc -triple=amdgcn -mcpu=gfx1100 -mattr=+wavefrontsize64,-real-true16 %s 2>&1 > /dev/null | FileCheck --check-prefix=W64-ERR --implicit-check-not=error: %s
----------------
kosarev wrote:

I was previously advised to use `-filetype=null` instead of `> /dev/null`, so maybe do that here as well for consistency?

Also, it seems the preference is to use `-comment-column=0` as we do in `gfx12_asm_vop1.s` to remove the extra spacing before `; encoding: ...`.

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


More information about the llvm-commits mailing list