Skip to content

Ignore non-existent refs in the transformed DWARF#149

Merged
sunfishcode merged 1 commit intobytecodealliance:masterfrom
yurydelendik:fix-bad-refs
Jun 26, 2019
Merged

Ignore non-existent refs in the transformed DWARF#149
sunfishcode merged 1 commit intobytecodealliance:masterfrom
yurydelendik:fix-bad-refs

Conversation

@yurydelendik
Copy link
Contributor

When used with the '-g' flag, wasmtime fails on more complex file. We don't have robust detection of non-used data so we just rely on DW_AT_ranges and DW_AT_low_pc attributes. It is possible we are loosing some of the info. But for now we want wasmtime not to fail -- the patch addresses the failure to "link" to the dropped entries.

@yurydelendik
Copy link
Contributor Author

/cc @SanderSpies (test file SanderSpies/ocaml@b71f011)

@sunfishcode
Copy link
Member

Looks good.

@sunfishcode sunfishcode merged commit 19cfec6 into bytecodealliance:master Jun 26, 2019
grishasobol pushed a commit to grishasobol/wasmtime that referenced this pull request Nov 29, 2021
* doc(host) Fix the `Externals` example

The example is missing two things:

  * `index` is computed but not used,
  * `check_signature` is never used.

This patch tries to fix that.

* doc(host) Fix `check_signature` example
pchickey added a commit to pchickey/wasmtime that referenced this pull request May 16, 2023
…es_and_nonblocking_streams

Eliminate nonblocking from descriptor flags
avanhatt pushed a commit to wellesley-prog-sys/wasmtime that referenced this pull request Oct 18, 2024
Adds a basic status tool that reports:

* Number of expansions in the coverage set (excluding skipped tags)
* Number of expansions in the coverage set that have specifications for
all terms
* Unspecified terms ranked by appearance count in the coverage set

Updates avanhatt#47
dicej added a commit to dicej/wasmtime that referenced this pull request May 8, 2025
…ix-146

remove unnecessary assertion from `subtask_cancel`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants