[Object][WebAssembly] Update format of error messages

sbc100 · sbc100 · commit 84c6f325840d · 2021-01-25T21:12:53.000-08:00
Error message should start with lowercase in accordance with https://llvm.org/docs/CodingStandards.html#error-and-warning-messages Differential Revision: https://reviews.llvm.org/D95239
diff --git a/lld/test/wasm/corrupted.wasm.test b/lld/test/wasm/corrupted.wasm.test
diff --git a/llvm/lib/Object/WasmObjectFile.cpp b/llvm/lib/Object/WasmObjectFile.cpp
diff --git a/llvm/test/Object/wasm-bad-metadata-version.yaml b/llvm/test/Object/wasm-bad-metadata-version.yaml
@@ -8,4 +8,4 @@ Sections:
     Name:            linking
     Version:         0
 
-# CHECK: {{.*}}: Unexpected metadata version: 0 (Expected: 2)
+# CHECK: {{.*}}: unexpected metadata version: 0 (Expected: 2)
diff --git a/llvm/test/Object/wasm-bad-reloc-type.test b/llvm/test/Object/wasm-bad-reloc-type.test
@@ -1,3 +1,3 @@
 RUN: not llvm-objdump -s %p/Inputs/WASM/bad-reloc-type.wasm 2>&1 | FileCheck %s
 
-CHECK: Bad relocation type: 63
+CHECK: invalid relocation type: 63
diff --git a/llvm/test/Object/wasm-bad-symbol-type.test b/llvm/test/Object/wasm-bad-symbol-type.test
@@ -1,3 +1,3 @@
 RUN: not llvm-objdump -s %p/Inputs/WASM/bad-symbol-type.wasm 2>&1 | FileCheck %s
 
-CHECK: Invalid symbol type: 63
+CHECK: invalid symbol type: 63
diff --git a/llvm/test/Object/wasm-duplicate-name.test b/llvm/test/Object/wasm-duplicate-name.test
@@ -26,4 +26,4 @@ Sections:
         Name:          b
 ...
 
-# CHECK: {{.*}}: Function named more than once
+# CHECK: {{.*}}: function named more than once
diff --git a/llvm/test/Object/wasm-invalid-file.yaml b/llvm/test/Object/wasm-invalid-file.yaml
@@ -13,10 +13,10 @@ FileHeader:
 # Append a section size that extends beyond the end of the file
 # RUN: not llvm-objdump -h %t.wasm 2>&1 | FileCheck %s -check-prefix=CHECK-SECTION-SIZE
 
-# CHECK-SECTION-SIZE: '{{.*}}.wasm': Section too large
+# CHECK-SECTION-SIZE: '{{.*}}.wasm': section too large
 
 # RUN: yaml2obj %s -o %t.wasm
 # # Append an section with invalid type (type 0x20, size 0x1, content 0x0)
 # RUN: echo -e -n "\x20\x01\x00" >> %t.wasm
 # RUN: not llvm-objdump -h %t.wasm 2>&1 | FileCheck %s -check-prefix=CHECK-SECTION-TYPE
-# CHECK-SECTION-TYPE: '{{.*}}.wasm': Invalid section type: 32
+# CHECK-SECTION-TYPE: '{{.*}}.wasm': invalid section type: 32
diff --git a/llvm/test/Object/wasm-invalid-section-order.test b/llvm/test/Object/wasm-invalid-section-order.test
@@ -1,5 +1,5 @@
 # RUN: not obj2yaml %p/Inputs/WASM/invalid-section-order.wasm 2>&1 | FileCheck %s
-# CHECK: {{.*}}: Out of order section type: 10
+# CHECK: {{.*}}: out of order section type: 10
 
 # Inputs/WASM/invalid-section-order.wasm is generated from this ll file, by
 # modifying WasmObjectWriter to incorrectly write the data section before the
diff --git a/llvm/test/Object/wasm-invalid-start.test b/llvm/test/Object/wasm-invalid-start.test
@@ -7,4 +7,4 @@ Sections:
   - Type:            START
     StartFunction:   0
 
-# CHECK: {{.*}}: Invalid start function
+# CHECK: {{.*}}: invalid start function
diff --git a/llvm/test/Object/wasm-missing-version.test b/llvm/test/Object/wasm-missing-version.test
@@ -1,2 +1,2 @@
 # RUN: not llvm-objdump -h %p/Inputs/WASM/missing-version.wasm 2>&1 | FileCheck %s
-# CHECK: {{.*}}: Missing version number
+# CHECK: {{.*}}: missing version number
diff --git a/llvm/test/ObjectYAML/wasm/header_invalid_version.yaml b/llvm/test/ObjectYAML/wasm/header_invalid_version.yaml
@@ -3,4 +3,4 @@
 FileHeader:
   Version:         0x00000002
 ...
-# CHECK: Error reading file: <stdin>: Bad version number
+# CHECK: Error reading file: <stdin>: invalid version number: 2
diff --git a/llvm/test/ObjectYAML/wasm/invalid_export.yaml b/llvm/test/ObjectYAML/wasm/invalid_export.yaml
@@ -10,4 +10,4 @@ Sections:
         Kind:            FUNCTION
         Index:           0x00000001
 
-# CHECK: Error reading file: <stdin>: Invalid function export
+# CHECK: Error reading file: <stdin>: invalid function export
diff --git a/llvm/test/ObjectYAML/wasm/producers_section_repeat.yaml b/llvm/test/ObjectYAML/wasm/producers_section_repeat.yaml
@@ -11,4 +11,4 @@ Sections:
       - Name:        C++
         Version:     C++17
 ...
-# CHECK:  Producers section contains repeated producer
+# CHECK: producers section contains repeated producer
diff --git a/llvm/test/tools/llvm-objdump/wasm/wasm-corrupt-section.test b/llvm/test/tools/llvm-objdump/wasm/wasm-corrupt-section.test
@@ -1,2 +1,2 @@
 # RUN: not llvm-objdump -h %p/Inputs/corrupt-section.wasm 2>&1 | FileCheck %s
-# CHECK: '{{.*}}corrupt-section.wasm': Section too large
+# CHECK: '{{.*}}corrupt-section.wasm': section too large
diff --git a/llvm/test/tools/llvm-readobj/wasm/wasm-invalid.test b/llvm/test/tools/llvm-readobj/wasm/wasm-invalid.test
@@ -4,4 +4,4 @@
 FileHeader:
   Version:         0x0000000c
 
-# CHECK: error: '<stdin>': Bad version number
+# CHECK: error: '<stdin>': invalid version number: 12

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`RUN: not llvm-objdump -s %p/Inputs/WASM/bad-reloc-type.wasm 2>&1 \| FileCheck %s`
`2`	`2`
`3`		`-CHECK: Bad relocation type: 63`
	`3`	`+CHECK: invalid relocation type: 63`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`RUN: not llvm-objdump -s %p/Inputs/WASM/bad-symbol-type.wasm 2>&1 \| FileCheck %s`
`2`	`2`
`3`		`-CHECK: Invalid symbol type: 63`
	`3`	`+CHECK: invalid symbol type: 63`
Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,2 @@`
`1`	`1`	`# RUN: not llvm-objdump -h %p/Inputs/WASM/missing-version.wasm 2>&1 \| FileCheck %s`
`2`		`-# CHECK: {{.*}}: Missing version number`
	`2`	`+# CHECK: {{.*}}: missing version number`