afonso360 opened PR #4430 from test-tables
to main
:
:wave: Hey,
As a precursor to adding table support to the interpreter this PR adds support for tables to the runtest environment.
This allows run tests to request memory for tables and for it to be provided via
vmctx
.
afonso360 created PR review comment:
This is a behaviour change from what we had before.
We used to put the start and end pointers. However it looks like that's not whats expected by the
bound
mechanism for heaps.This changes it to pass the size in bytes of the heap.
afonso360 submitted PR review.
afonso360 submitted PR review.
afonso360 created PR review comment:
This is wrong according to
docs/ir.md
.Table bounds should be the number of elements and not the number of bytes.
However nothing works when I change it to that. Probably something to investigate further.
afonso360 closed without merge PR #4430.
Last updated: Dec 23 2024 at 12:05 UTC