Fixed a typo in the test s/CEHCK/CHECK/

[gribozavr]

Summary

Turns out the test was not correct, I had to adjust the test to work. I
also added CHECK-LABELs for better error messages from FileCheck while
I'm here.

[jsji]

LGTM. Thanks for catching this, I am surprised that those typos slipped through ... FYI. @HLJ2009

[lebedev.ri]

In D58614#1408924, @jsji wrote:

LGTM. Thanks for catching this, I am surprised that those typos slipped through ... FYI. @HLJ2009

Two things here.

1. Why is CHECK-NEXT not being used?
2. Why is ./utils/update_llc_test_checks.py not being used? That dramatically reduces chances of things like this, and spares one from actually having to manually write proper check lines. (thus improves test coverage. one will of course still need to verify that the checks are correct.)

[jsji]

Why is CHECK-NEXT not being used?
Why is ./utils/update_llc_test_checks.py not being used? That dramatically reduces chances of things like this, and spares one from actually having to manually write proper check lines. (thus improves test coverage. one will of course still need to verify that the checks are correct.)

Good questions, I think this was tricky and it should be due to "\0A" in the inline asm here.
The output will have empty lines after each assembly line.

#APP
xxsldwi vs0, v2, v2, 3

xscvspdp f0, f0

fctiw f0, f0

mffprd  r3, f0


#NO_APP

So, we can NOT use CHECK-NEXT,
and the script ./utils/update_llc_test_checks.py also can NOT handle it well either:
the output of scripts will have empty string CHECK-NEXT, then causing failures.

inlineasm-vsx-reg.ll:8:15: error: found empty check string with prefix 'CHECK:'
; CHECK-NEXT: 
              ^

But agree that we should recommend using CHECK-NEXT and ./utils/update_llc_test_checks.py if possible.

[HLJ2009]

In D58614#1408924, @jsji wrote:

LGTM. Thanks for catching this, I am surprised that those typos slipped through ... FYI. @HLJ2009

Sorry for this error, I am very grateful to point this out.

llvm/test/CodeGen/PowerPC/inlineasm-vsx-reg.ll

 ; RUN: llc -verify-machineinstrs -ppc-vsr-nums-as-vr -ppc-asm-full-reg-names  -mcpu=pwr9 -mtriple=powerpc64le-unknown-unknown < %s | FileCheck %s
 define signext i32 @foo(<4 x float> %__A) {
 entry:
   %0 = tail call { i32, <4 x float> } asm "xxsldwi ${1:x},${2:x},${2:x},3;\0Axscvspdp ${1:x},${1:x};\0Afctiw  $1,$1;\0Amfvsrd  $0,${1:x};\0A", "=r,=&^wa,^wa"(<4 x float> %__A)
   %asmresult = extractvalue { i32, <4 x float> } %0, 0
   ret i32 %asmresult
+; CHECK-LABEL: foo:
 ; CHECK: #APP
 ; CHECK: xxsldwi vs0, v2, v2, 3
 ; CHECK: xscvspdp f0, f0
 ; CHECK: fctiw f0, f0
 ; CHECK: mffprd r3, f0
 ; CHECK: #NO_APP
 }
 
 define signext i32 @foo1(<4 x float> %__A) {
 entry:
   %0 = tail call { i32, <4 x float> } asm "xxsldwi ${1:x},${2:x},${2:x},3;\0Axscvspdp ${1:x},${1:x};\0Afctiw  $1,$1;\0Amfvsrd  $0,${1:x};\0A", "=r,=&^wi,^wa"(<4 x float> %__A)
   %asmresult = extractvalue { i32, <4 x float> } %0, 0
   ret i32 %asmresult
 
+; CHECK-LABEL: foo1:
 ; CHECK: #APP
 ; CHECK: xxsldwi vs0, v2, v2, 3
 ; CHECK: xscvspdp f0, f0
-; CEHCK: fctiw f0, f0
+; CHECK: fctiw f0, f0
 ; CHECK: mffprd r3, f0
-; CEHCK: extsw r3, r3
 ; CHECK: #NO_APP
+; CHECK: extsw r3, r3
 }
 
 define double @test() {
   entry:
     %0 = tail call double asm "mtvsrd ${0:x}, 1", "=^ws,~{f0},~{f1},~{f2},~{f3},~{f4},~{f5},~{f6},~{f7},~{f8},~{f9},~{f10},~{f11},~{f12},~{f13},~{f14}"()
     ret double %0
+; CHECK-LABEL: test:
 ; CHECK: #APP
 ; CHECK: mtvsrd v2, r1
 ; CHECK: #NO_APP
 }