[PATCH] D106897: [lld][WebAssembly] Prefer objdump -d over obj2yaml for tests. NFC

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 27 11:08:30 PDT 2021


sbc100 created this revision.
Herald added subscribers: wingo, ecnelises, sunfish, jgravelle-google, dschuff.
sbc100 requested review of this revision.
Herald added subscribers: llvm-commits, aheejin.
Herald added a project: LLVM.

Now that we have https://reviews.llvm.org/D105539 we can
use objdump -d to actually check for instruction sequences
rather than binary blobs.

This is just an example of how to do that we should followup
with a wider ranging conversion of existing tests.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D106897

Files:
  lld/test/wasm/startstop.ll


Index: lld/test/wasm/startstop.ll
===================================================================
--- lld/test/wasm/startstop.ll
+++ lld/test/wasm/startstop.ll
@@ -2,6 +2,7 @@
 ; RUN: llc -filetype=obj %p/Inputs/explicit-section.ll -o %t2.o
 ; RUN: wasm-ld --export=get_start --export=get_end --export=foo --export=var1 %t.o %t2.o -o %t.wasm
 ; RUN: obj2yaml %t.wasm | FileCheck %s
+; RUN: llvm-objdump -d --no-show-raw-insn %t.wasm | FileCheck %s --check-prefix=ASM
 
 target triple = "wasm32-unknown-unknown"
 
@@ -23,18 +24,8 @@
 entry:
   ret void
 }
-; CHECK:        - Type:            CODE
-; CHECK-NEXT:     Functions:
-; CHECK-NEXT:       - Index:           0
-; CHECK-NEXT:         Locals:          []
-; CHECK-NEXT:         Body:            4180888080000B
-; CHECK-NEXT:       - Index:           1
-; CHECK-NEXT:         Locals:          []
-; CHECK-NEXT:         Body:            4190888080000B
-; CHECK-NEXT:       - Index:           2
-; CHECK-NEXT:         Locals:          []
-; CHECK-NEXT:         Body:            0B
-; CHECK-NEXT:   - Type:            DATA
+
+;      CHECK:   - Type:            DATA
 ; CHECK-NEXT:     Segments:
 ; CHECK-NEXT:       - SectionOffset:   7
 ; CHECK-NEXT:         InitFlags:       0
@@ -42,12 +33,13 @@
 ; CHECK-NEXT:           Opcode:          I32_CONST
 ; CHECK-NEXT:           Value:           1024
 ; CHECK-NEXT:         Content:         03000000040000002A0000002B000000
-; CHECK-NEXT:   - Type:            CUSTOM
-; CHECK-NEXT:     Name:            name
-; CHECK-NEXT:     FunctionNames:
-; CHECK-NEXT:       - Index:           0
-; CHECK-NEXT:         Name:            get_start
-; CHECK-NEXT:       - Index:           1
-; CHECK-NEXT:         Name:            get_end
-; CHECK-NEXT:       - Index:           2
-; CHECK-NEXT:         Name:            _start
+
+;       ASM: 00000001 <get_start>:
+; ASM-EMPTY:
+;  ASM-NEXT:        3:     i32.const 1024
+;  ASM-NEXT:        9:     end
+
+;       ASM: 0000000a <get_end>:
+; ASM-EMPTY:
+;  ASM-NEXT:        c:     i32.const 1040
+;  ASM-NEXT:       12:     end


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106897.362107.patch
Type: text/x-patch
Size: 2068 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210727/cdef7aa5/attachment.bin>


More information about the llvm-commits mailing list