github-actions[bot] commented on Issue #2322:
Subscribe to Label Action
cc @peterhuene
<details>
This issue or pull request has been labeled: "wasmtime:api"Thus the following users have been cc'd because of the following labels:
- peterhuene: wasmtime:api
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
fitzgen commented on Issue #2322:
To be clear this is just enabling variable length encoding for integers, right? It is not switching to a delta encoding (which would ensure that essentially all addresses are 1-byte encoded) correct?
alexcrichton commented on Issue #2322:
Ah yes indeed, a delta encoding might make it even more further compact.
I've so far shied away from big changes like that though. I think we may want to do it eventually, but it trades-off in-memory footprint for lookup speed since we can no longer quickly do a random lookup for a particular trapping pc. Not that we necessarily need that to be too too fast.
fitzgen commented on Issue #2322:
I'm only advocating for delta encoding in the serialized format, not the in-memory representation.
tschneidereit commented on Issue #2322:
I'm only advocating for delta encoding in the serialized format, not the in-memory representation.
I think it'd be good to try not to have those two deviate too much: ideally we should at some point be able to mmap the on-disk representation entirely or almost entirely instead of having to read and deserialize it.
There clearly are size/speed tradeoffs to be had here, so that might not be the only thing we support, but I hope it can become a thing we support :)
Last updated: Dec 23 2024 at 13:07 UTC