Stream: git-wasmtime

Topic: wasmtime / PR #9902 Add `WASMTIME_LOG_NO_CONTEXT`

view this post on Zulip Wasmtime GitHub notifications bot (Dec 25 2024 at 20:10):

SingleAccretion opened PR #9902 from SingleAccretion:Log-NoCtx to bytecodealliance:main:

When using trace logging for compiler debugging, the usual 'context': time, level, target, are just visual noise.

Add an environment variable to disable them. There is no change to default behavior.

Sample output diff:

- 2024-12-25T20:07:32.459831Z TRACE debug-info-transform: === Begin DIE at 0x0000229a (depth = 2):
+ === Begin DIE at 0x0000229a (depth = 2):
  DW_TAG_GNU_call_site
    DW_AT_abstract_origin (0x000022a4)
    DW_AT_low_pc (ffffffff)

view this post on Zulip Wasmtime GitHub notifications bot (Dec 25 2024 at 20:13):

SingleAccretion updated PR #9902.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 25 2024 at 20:16):

SingleAccretion edited PR #9902:

When using trace logging for compiler debugging, the usual 'context': time, level, target, are just visual noise and take up horizontal space.

Add an environment variable to disable them. There is no change to default behavior.

Sample output diff:

- 2024-12-25T20:07:32.459831Z TRACE debug-info-transform: === Begin DIE at 0x0000229a (depth = 2):
+ === Begin DIE at 0x0000229a (depth = 2):
  DW_TAG_GNU_call_site
    DW_AT_abstract_origin (0x000022a4)
    DW_AT_low_pc (ffffffff)

view this post on Zulip Wasmtime GitHub notifications bot (Dec 25 2024 at 20:33):

SingleAccretion has marked PR #9902 as ready for review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 25 2024 at 20:33):

SingleAccretion requested fitzgen for a review on PR #9902.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 25 2024 at 20:33):

SingleAccretion requested wasmtime-core-reviewers for a review on PR #9902.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 25 2024 at 20:33):

SingleAccretion requested wasmtime-default-reviewers for a review on PR #9902.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 26 2024 at 16:05):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 26 2024 at 16:05):

pchickey created PR review comment:

Just calling out this change as unrelated - we can merge it in this PR. I don’t understand cranelift well enough to review it properly but since you seem to know this stuff inside out I’m fine with it.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 26 2024 at 16:25):

pchickey merged PR #9902.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 26 2024 at 19:50):

SingleAccretion submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 26 2024 at 19:50):

SingleAccretion created PR review comment:

Thanks!

This is a bugfix for the debug logging you can turn on to see how DIEs get processed - it now prints the section (global) offsets for all referenced DIEs, allowing one to easily search for them in the editor.

Last updated: Jan 24 2025 at 00:11 UTC