Stream: git-wasmtime

Topic: wasmtime / PR #4824 cranelift: Generate loads and stores ...

Wasmtime GitHub notifications bot (Aug 30 2022 at 19:01):

afonso360 opened PR #4824 from fuzz-amodes to main:

:wave: Hey,

This PR adds loads and stores to fuzzgen.

The general strategy is to get a address from one of many sources (stack_addr/heap_addr/etc...), add some offsets, add more instructions to modify the address (to try to get the backends to fuse multiple instructions into a single load/store) and then emit the load/store.

This is a start to the above. We only support stack_addr and don't insert any amode instructions.

Wasmtime GitHub notifications bot (Aug 30 2022 at 19:15):

afonso360 edited PR #4824 from fuzz-amodes to main:

:wave: Hey,

This PR adds loads and stores to fuzzgen.

The general strategy is to get a address from one of many sources (stack_addr/heap_addr/etc...), add some offsets, add more instructions to modify the address (to try to get the backends to fuse multiple instructions into a single load/store) and then emit the load/store.

This is a start to the above. We only support stack_addr and don't insert any amode instructions.

I'm still a bit unsure on what is the best way to generate the amode instructions, which opcodes should we support (I guess we should check what the backends match right now), and how to ensure that the addresses remain somewhat reasonable.

Valid addresses aren't necessary, but I'd prefer to avoid wasting runs on iadd_imm addr, 0xffffffff or something like that.

Wasmtime GitHub notifications bot (Aug 31 2022 at 22:20):

jameysharp submitted PR review.

Wasmtime GitHub notifications bot (Sep 07 2022 at 18:00):

jameysharp merged PR #4824.

Last updated: Oct 23 2024 at 20:03 UTC