-
Notifications
You must be signed in to change notification settings - Fork 149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make infiniteGas
default, add setGas
cheatcode
#2045
Conversation
dd43898
to
c9be2f3
Compare
02d6f5c
to
8583738
Compare
036824d
to
0c0c458
Compare
2a41832
to
ede0758
Compare
43d91db
to
553b84a
Compare
This PR should be migrated to the new Kontrol repo: runtimeverification/kontrol, instead. |
@ehildenb Kontrol-specific changes have been moved to runtimeverification/kontrol#57. This PR includes changes to |
Closed in favor of the Kontrol PR (runtimeverification/kontrol#57) and a smaller |
Closes runtimeverification/kontrol#38 and #2058 (infinite gas prevents
setUp
from branching on potential out-of-gas caused bySSTORE
-ing a symbolic value).This PR makes the following changes:
kevm.setGas(uint256)
cheatcode that allows setting the available gas; adds a test for it (GasTest.testSetGas
);kevm.infiniteGas()
from tests, as it's no longer necessary;test_foundry_merge_nodes
,LoopsTest.sum_N
and the associated loop invariant accordingly (adding or removing gas-related cheatcodes changes program counter values and node numbering, which are hardcoded in these two tests).