fitzgen edited PR #7374:
In this case, we don't need to load the dynamic heap bound from the vmctx
because it actually has a constant size. Instead, we can use the constant
directly.
alexcrichton submitted PR review:
Mind adding a test for a
(memory 1 2)
as well to ensure that a constant size isn't used?
fitzgen submitted PR review.
fitzgen created PR review comment:
Might be better to
and_then
instead ofmap
here actually and not swap out the max foru64::MAX
on overflow?
fitzgen submitted PR review.
fitzgen submitted PR review.
fitzgen created PR review comment:
Done.
fitzgen updated PR #7374.
fitzgen has enabled auto merge for PR #7374.
fitzgen merged PR #7374.
Last updated: Dec 23 2024 at 12:05 UTC