Stream: git-wasmtime

Topic: wasmtime / Issue #1618 Cranelift aarch64: support heaps w...

view this post on Zulip Wasmtime GitHub notifications bot (Apr 28 2020 at 12:37):

bnjbvr opened Issue #1618:

The new backend supports heaps with implicit bounds check (using the large virtual memory allocation + segfault trick), but doesn't support generating explicit bounds checks when we don't want large virtual memory allocations (or can't do them, b/o containerization, etc.).

Currently, an assertion triggers during lowering, when running for instance Spidermonkey's memory.js test case with --disable-wasm-huge-memory:

Hit MOZ_CRASH(called `Option::unwrap()` on a `None` value) at /home/ben/code/wasmtime/cranelift/codegen/src/isa/aarch64/lower.rs:1894

cc @cfallin

view this post on Zulip Wasmtime GitHub notifications bot (Apr 28 2020 at 12:37):

bnjbvr labeled Issue #1618:

The new backend supports heaps with implicit bounds check (using the large virtual memory allocation + segfault trick), but doesn't support generating explicit bounds checks when we don't want large virtual memory allocations (or can't do them, b/o containerization, etc.).

Currently, an assertion triggers during lowering, when running for instance Spidermonkey's memory.js test case with --disable-wasm-huge-memory:

Hit MOZ_CRASH(called `Option::unwrap()` on a `None` value) at /home/ben/code/wasmtime/cranelift/codegen/src/isa/aarch64/lower.rs:1894

cc @cfallin

view this post on Zulip Wasmtime GitHub notifications bot (Apr 28 2020 at 12:37):

bnjbvr labeled Issue #1618:

The new backend supports heaps with implicit bounds check (using the large virtual memory allocation + segfault trick), but doesn't support generating explicit bounds checks when we don't want large virtual memory allocations (or can't do them, b/o containerization, etc.).

Currently, an assertion triggers during lowering, when running for instance Spidermonkey's memory.js test case with --disable-wasm-huge-memory:

Hit MOZ_CRASH(called `Option::unwrap()` on a `None` value) at /home/ben/code/wasmtime/cranelift/codegen/src/isa/aarch64/lower.rs:1894

cc @cfallin

view this post on Zulip Wasmtime GitHub notifications bot (Apr 28 2020 at 12:37):

github-actions[bot] commented on Issue #1618:

Subscribe to Label Action

cc @bnjbvr

<details>
This issue or pull request has been labeled: "cranelift"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

Last updated: Dec 23 2024 at 13:07 UTC