Stream: git-wasmtime

Topic: wasmtime / PR #53 Write .debug_frame information


view this post on Zulip Wasmtime GitHub notifications bot (Mar 09 2020 at 21:48):

yurydelendik updated PR #53 from debug-frame to master:

Requires https://github.com/CraneStation/cranelift/pull/679 and https://github.com/CraneStation/wasmtime/pull/50

There is a problem when lldb is using wasmtime's frame information -- the JIT's debug frame information a) is not present, and b) debugger confused about which info to used.

This patch generates .debug_frame data for each function wasmtime generates. It can be added to the object file.

See also the patch for lldb to force using JIT's .debug_frame can be found at https://gist.github.com/yurydelendik/12e95a8eb474cc72b6f57a5e23091416

view this post on Zulip Wasmtime GitHub notifications bot (Mar 10 2020 at 13:18):

yurydelendik updated PR #53 from debug-frame to master:

Requires https://github.com/CraneStation/cranelift/pull/679 and https://github.com/CraneStation/wasmtime/pull/50

There is a problem when lldb is using wasmtime's frame information -- the JIT's debug frame information a) is not present, and b) debugger confused about which info to used.

This patch generates .debug_frame data for each function wasmtime generates. It can be added to the object file.

See also the patch for lldb to force using JIT's .debug_frame can be found at https://gist.github.com/yurydelendik/12e95a8eb474cc72b6f57a5e23091416

view this post on Zulip Wasmtime GitHub notifications bot (Mar 10 2020 at 13:22):

yurydelendik updated PR #53 from debug-frame to master:

Requires https://github.com/CraneStation/cranelift/pull/679 and https://github.com/CraneStation/wasmtime/pull/50

There is a problem when lldb is using wasmtime's frame information -- the JIT's debug frame information a) is not present, and b) debugger confused about which info to used.

This patch generates .debug_frame data for each function wasmtime generates. It can be added to the object file.

See also the patch for lldb to force using JIT's .debug_frame can be found at https://gist.github.com/yurydelendik/12e95a8eb474cc72b6f57a5e23091416

view this post on Zulip Wasmtime GitHub notifications bot (Mar 10 2020 at 13:35):

yurydelendik updated PR #53 from debug-frame to master:

Requires https://github.com/CraneStation/cranelift/pull/679 and https://github.com/CraneStation/wasmtime/pull/50

There is a problem when lldb is using wasmtime's frame information -- the JIT's debug frame information a) is not present, and b) debugger confused about which info to used.

This patch generates .debug_frame data for each function wasmtime generates. It can be added to the object file.

See also the patch for lldb to force using JIT's .debug_frame can be found at https://gist.github.com/yurydelendik/12e95a8eb474cc72b6f57a5e23091416

view this post on Zulip Wasmtime GitHub notifications bot (Mar 10 2020 at 14:47):

yurydelendik updated PR #53 from debug-frame to master:

Requires https://github.com/CraneStation/cranelift/pull/679 and https://github.com/CraneStation/wasmtime/pull/50

There is a problem when lldb is using wasmtime's frame information -- the JIT's debug frame information a) is not present, and b) debugger confused about which info to used.

This patch generates .debug_frame data for each function wasmtime generates. It can be added to the object file.

See also the patch for lldb to force using JIT's .debug_frame can be found at https://gist.github.com/yurydelendik/12e95a8eb474cc72b6f57a5e23091416

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 13:12):

bjorn3 submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 13:12):

bjorn3 submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 13:12):

bjorn3 created PR Review Comment:

Is this supposed to be negative?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 13:12):

bjorn3 created PR Review Comment:

        // FIXME Can only process functions with SystemV-like prologue.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 13:49):

yurydelendik updated PR #53 from debug-frame to master:

Requires https://github.com/CraneStation/cranelift/pull/679 and https://github.com/CraneStation/wasmtime/pull/50

There is a problem when lldb is using wasmtime's frame information -- the JIT's debug frame information a) is not present, and b) debugger confused about which info to used.

This patch generates .debug_frame data for each function wasmtime generates. It can be added to the object file.

See also the patch for lldb to force using JIT's .debug_frame can be found at https://gist.github.com/yurydelendik/12e95a8eb474cc72b6f57a5e23091416

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 13:53):

yurydelendik submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 13:53):

yurydelendik created PR Review Comment:

Since stack grow in "negative" direction: 8 saves on encoding of offset LEB size and the sign indicates direction.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 11 2020 at 15:22):

yurydelendik merged PR #53.


Last updated: Oct 23 2024 at 20:03 UTC