Skip to content

[WIP] feat: _exceptional halts_ in AVM simulator (like div/0) consume all remaining allocated gas#10259

Closed
dbanks12 wants to merge 1 commit intomasterfrom
db/charge-all-gas-on-exceptional-halts
Closed

[WIP] feat: _exceptional halts_ in AVM simulator (like div/0) consume all remaining allocated gas#10259
dbanks12 wants to merge 1 commit intomasterfrom
db/charge-all-gas-on-exceptional-halts

Conversation

@dbanks12
Copy link
Contributor

No description provided.

@dbanks12 dbanks12 requested a review from fcarreiro as a code owner November 27, 2024 20:32
@dbanks12 dbanks12 changed the title feat_exceptional halts_ in AVM simulator (like div/0) consume all remaining allocated gas feat: _exceptional halts_ in AVM simulator (like div/0) consume all remaining allocated gas Nov 27, 2024
Copy link
Contributor

@fcarreiro fcarreiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG but is this in sync with witgen?

@dbanks12
Copy link
Contributor Author

LG but is this in sync with witgen?

... nope! Going to figure out how to do the witgen side of it before we merge. Will flag WIP

@dbanks12 dbanks12 changed the title feat: _exceptional halts_ in AVM simulator (like div/0) consume all remaining allocated gas [WIP] feat: _exceptional halts_ in AVM simulator (like div/0) consume all remaining allocated gas Nov 27, 2024
Copy link
Contributor

I wouldn't be surprised if you need to disable some constraint (or modify it to accomodate the jump in gas unrelated to the cost of the opcode)

@dbanks12 dbanks12 marked this pull request as draft December 11, 2024 01:18
@dbanks12
Copy link
Contributor Author

Replaced by #10731

@dbanks12 dbanks12 closed this Dec 17, 2024
@dbanks12 dbanks12 deleted the db/charge-all-gas-on-exceptional-halts branch April 21, 2025 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants