pnkfelix / krabcake

Umbrella repository for Krabcake experiments
Other
46 stars 1 forks source link

fix bug in valgrind_do_client_request_expr #20

Closed pnkfelix closed 1 year ago

pnkfelix commented 1 year ago

client requests communicate via the rdx register not rdi.

pnkfelix commented 1 year ago

r? @bryangarza

bryangarza commented 1 year ago

@pnkfelix can you add a new UI test with the printing that triggered the bug?

pnkfelix commented 1 year ago

It’s tricky to do it properly; exposing the bug relies on how the compiler chooses to b optimize the code

but I have an idea of a couple ways to do it