Skip to content

[PowerPC] fixed mtvsrbmi.ll test case error caused by run the update_llc_test_checks.py #144075

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 13, 2025

Conversation

diggerlin
Copy link
Contributor

fixed mtvsrbmi.ll test case error which caused by run the update_llc_test_checks.py

@llvmbot
Copy link
Member

llvmbot commented Jun 13, 2025

@llvm/pr-subscribers-backend-powerpc

Author: zhijian lin (diggerlin)

Changes

fixed mtvsrbmi.ll test case error which caused by run the update_llc_test_checks.py


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

1 Files Affected:

  • (modified) llvm/test/CodeGen/PowerPC/mtvsrbmi.ll (+8-14)
diff --git a/llvm/test/CodeGen/PowerPC/mtvsrbmi.ll b/llvm/test/CodeGen/PowerPC/mtvsrbmi.ll
index 7ed57c300ec71..5486dc02faf90 100644
--- a/llvm/test/CodeGen/PowerPC/mtvsrbmi.ll
+++ b/llvm/test/CodeGen/PowerPC/mtvsrbmi.ll
@@ -10,15 +10,6 @@
 ; RUN:   | FileCheck %s --check-prefix=CHECK
 
 define dso_local noundef range(i8 -1, 1) <16 x i8> @_Z5v00FFv() {
-; CHECK-LABEL: _Z5v00FFv:
-; CHECK:       # %bb.0: # %entry
-; CHECK-NEXT:    lwz r3, L..C0(r2) # %const.0
-; CHECK-NEXT:    lxv vs34, 0(r3)
-; CHECK-NEXT:    blr
-entry:
-  ret <16 x i8> <i8 -1, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0>
-}
-
 ; CHECK:      L..CPI0_0:
 ; CHECK-NEXT:   .byte   255                             # 0xff
 ; CHECK-NEXT:   .byte   0                               # 0x0
@@ -37,8 +28,11 @@ entry:
 ; CHECK-NEXT:   .byte   0                               # 0x0
 ; CHECK-NEXT:   .byte   0                               # 0x0
 
-; CHECK:      ._Z5v00FFv:
-; CHECK-NEXT: # %bb.0:
-; CHECK-NEXT:   lwz r3, L..C0(r2)
-; CHECK-NEXT:   lxv vs34, 0(r3)
-; CHECK-NEXT:   blr
+; CHECK-LABEL: _Z5v00FFv:
+; CHECK:       # %bb.0: # %entry
+; CHECK-NEXT:    lwz r3, L..C0(r2) # %const.0
+; CHECK-NEXT:    lxv vs34, 0(r3)
+; CHECK-NEXT:    blr
+entry:
+  ret <16 x i8> <i8 -1, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0, i8 0>
+}

Copy link
Contributor

@lei137 lei137 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

guess the tool doesn't deal will with const pools.
LGTM.

@diggerlin diggerlin merged commit ea73fc5 into llvm:main Jun 13, 2025
7 of 9 checks passed
tomtor pushed a commit to tomtor/llvm-project that referenced this pull request Jun 14, 2025
…llc_test_checks.py (llvm#144075)

fixed mtvsrbmi.ll test case error which caused by run the
update_llc_test_checks.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants