[llvm] [AMDGPU][RFC] Combine asm and disasm tests. (PR #92895)
Ivan Kosarev via llvm-commits
llvm-commits at lists.llvm.org
Thu May 30 07:03:22 PDT 2024
================
@@ -1,3590 +1,3597 @@
-// RUN: llvm-mc -triple=amdgcn -mcpu=gfx1200 -mattr=+wavefrontsize32,-wavefrontsize64 -show-encoding %s | FileCheck --check-prefix=GFX12 %s
-// RUN: llvm-mc -triple=amdgcn -mcpu=gfx1200 -mattr=-wavefrontsize32,+wavefrontsize64 -show-encoding %s | FileCheck --check-prefix=GFX12 %s
+// RUN: llvm-mc -triple=amdgcn -mcpu=gfx1200 -mattr=+wavefrontsize32,-wavefrontsize64 -show-encoding %s | FileCheck --strict-whitespace --check-prefixes=GFX12,GFX12-ASM %s
+// RUN: llvm-mc -triple=amdgcn -mcpu=gfx1200 -mattr=+wavefrontsize32,-wavefrontsize64 -show-encoding %s | sed -n 's/.*encoding: \(\[.*\]\).*/\1/p' | llvm-mc -triple=amdgcn -mcpu=gfx1200 -mattr=+wavefrontsize32,-wavefrontsize64 -disassemble -show-encoding | FileCheck --strict-whitespace --check-prefixes=GFX12,GFX12-DIS %s
----------------
kosarev wrote:
> Does `s/.*encoding://p` work?
Yes!
> This version does increase the run line complexity quite a bit. We have to maintain this regex in every test, which isn't great.
Added a lit substitution to address that.
https://github.com/llvm/llvm-project/pull/92895
More information about the llvm-commits
mailing list