[llvm] [AMDGPU][test]update error dasm test for update-mc-test-check script (PR #111760)

via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 9 22:25:36 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-amdgpu

Author: Brox Chen (broxigarchen)

<details>
<summary>Changes</summary>

The previous error test line is using a 16bit instruction to indicate an error. However this is a poor pick. 

The 16bit instructions on AMDGPU is under development and thus, some downstream branches are not showing this exact error message. Changing it to another error dasm code.

---
Full diff: https://github.com/llvm/llvm-project/pull/111760.diff


2 Files Affected:

- (modified) llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt (+1-1) 
- (modified) llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt.expected (+1-1) 


``````````diff
diff --git a/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt b/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt
index 9f5fba6e50df25..e8338577cfc47c 100644
--- a/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt
+++ b/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt
@@ -2,4 +2,4 @@
 
 0x00,0x00,0x00,0x7e
 
-0xfd,0xb8,0x0a,0x7f
+0x00,0x00,0x00,0x01
diff --git a/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt.expected b/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt.expected
index 1b64695fc29408..a6f7abcb1774ac 100644
--- a/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt.expected
+++ b/llvm/test/tools/UpdateTestChecks/update_mc_test_checks/Inputs/amdgpu_dasm.txt.expected
@@ -4,5 +4,5 @@
 0x00,0x00,0x00,0x7e
 # CHECK: v_nop                                   ; encoding: [0x00,0x00,0x00,0x7e]
 
-0xfd,0xb8,0x0a,0x7f
+0x00,0x00,0x00,0x01
 # CHECK: :[[@LINE-1]]:1: warning: invalid instruction encoding

``````````

</details>


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


More information about the llvm-commits mailing list