afonso360 opened PR #4824 from fuzz-amodes
to main
:
:wave: Hey,
This PR adds
load
s andstore
s 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.
afonso360 edited PR #4824 from fuzz-amodes
to main
:
:wave: Hey,
This PR adds
load
s andstore
s 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.
jameysharp submitted PR review.
jameysharp merged PR #4824.
Last updated: Dec 23 2024 at 12:05 UTC