Stream: git-wasmtime

Topic: wasmtime / PR #8637 Display symbol names "compile -D"


view this post on Zulip Wasmtime GitHub notifications bot (May 21 2024 at 20:29):

lpereira edited PR #8637.

view this post on Zulip Wasmtime GitHub notifications bot (May 21 2024 at 20:30):

lpereira edited PR #8637:

Another one in the series to improve the debuggability quality of life: display the symbol name in the "compile -D" command.

No tests seems to expect anything with the string "Disassembly of {num} bytes", so no tests have been updated.

Here's an output sample (excerpt from compile -D cranelift/filetests/filetests/wasm/f64-arith.clif):

Disassembly of 44 bytes <%f64_copysign>:
   0:   55                      pushq   %rbp
   1:   48 89 e5                movq    %rsp, %rbp
   4:   48 b9 00 00 00 00 00 00 00 80
                movabsq $9223372036854775808, %rcx
   e:   66 48 0f 6e f9          movq    %rcx, %xmm7
  13:   66 0f 6f d0             movdqa  %xmm0, %xmm2
  17:   66 0f 6f c7             movdqa  %xmm7, %xmm0
  1b:   66 0f 55 c2             andnpd  %xmm2, %xmm0
  1f:   66 0f 54 f9             andpd   %xmm1, %xmm7
  23:   66 0f 56 c7             orpd    %xmm7, %xmm0
  27:   48 89 ec                movq    %rbp, %rsp
  2a:   5d                      popq    %rbp
  2b:   c3                      retq

view this post on Zulip Wasmtime GitHub notifications bot (May 21 2024 at 20:30):

lpereira commented on PR #8637:

Rebased and force-pushed with only the change to show the function name.

view this post on Zulip Wasmtime GitHub notifications bot (May 21 2024 at 21:10):

elliottt submitted PR review:

Looks great, thanks!

view this post on Zulip Wasmtime GitHub notifications bot (May 21 2024 at 21:26):

elliottt merged PR #8637.


Last updated: Dec 23 2024 at 13:07 UTC