Stream: git-wasmtime

Topic: wasmtime / PR #5335 Cranelift: consider heap's guard page...


view this post on Zulip Wasmtime GitHub notifications bot (Nov 28 2022 at 23:26):

fitzgen opened PR #5335 from heap-addr-guard-pages to main:

Fixes #5328

FYI, I opted not to mess with or dedupe the similar logic in cranelift-wasm since that stuff is going to go away soon-ish when we remove heap_addr and collect all this logic into heap_{load,store}. Excited to get to a point where this logic isn't spread across crates and straddling interfaces.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 28 2022 at 23:26):

fitzgen requested alexcrichton for a review on PR #5335.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 01:49):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 01:49):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 01:49):

cfallin created PR review comment:

Is it worth a comment here on the >= and the 0xffff_ffff (i.e., the minus-one aspect)?

In particular I think it's right because

I think it's a little unclear because above in the comment we have index < 4GiB <= RHS but here we are really relying on index <= (4GiB - 1) <= RHS, which is also valid (<= is also transitive). I guess aligning the two makes things clearer, at least for me...

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 15:17):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 19:21):

fitzgen updated PR #5335 from heap-addr-guard-pages to main.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 19:23):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 19:23):

fitzgen created PR review comment:

Updated the comment above this to align more directly with the implementation.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 19:41):

fitzgen has enabled auto merge for PR #5335.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 29 2022 at 19:54):

fitzgen merged PR #5335.


Last updated: Dec 23 2024 at 13:07 UTC