ethereum-optimism / superchain-ops

30 stars 28 forks source link

fix: skip precompile code check and silence compiler warnings #164

Closed mds1 closed 3 months ago

mds1 commented 3 months ago

cc @emilianobonassi, give this PR a try

emilianobonassi commented 3 months ago

it works

mds1 commented 3 months ago

Might be worth changing forge recording behavior to ignore precompiles for Accesses. Since they're more like pure functions.

This is an interesting point, worth thinking about. I think the current behavior is ok though and I'd rather lean towards the cheat returning a more complete picture of the trace, than a less complete one