Skip to content

Revert "fix(gasmeter): account for new-account and memory gas in TRON instructions"#126

Merged
CodeNinjaEvan merged 1 commit into
tronprotocol:release_0.8.28from
yanghang8612:revert/tron-gas-metering
Jun 29, 2026
Merged

Revert "fix(gasmeter): account for new-account and memory gas in TRON instructions"#126
CodeNinjaEvan merged 1 commit into
tronprotocol:release_0.8.28from
yanghang8612:revert/tron-gas-metering

Conversation

@yanghang8612

Copy link
Copy Markdown

Summary

Reverts the gas-metering changes introduced in #120 (commit abbdee3), restoring libevmasm/GasMeter.cpp to its prior state.

This reverts:

  • The two memoryGas(...) charges added to NATIVEVOTE.
  • Splitting NATIVEDELEGATERESOURCE / NATIVEUNDELEGATERESOURCE out of the shared Stake-2.0 group to add callNewAccountGas.

The six V2 opcodes are merged back into the single freezeV2Gas branch, and NATIVEVOTE is back to charging only voteGas.

Note: #120 only affected compile-time gas estimation in the optimizer, not runtime correctness — so this revert likewise has no effect on generated bytecode semantics.

Clean revert: no commit after #120 touched this file. solc builds cleanly.

Targets release_0.8.28.

🤖 Generated with Claude Code

@github-actions

Copy link
Copy Markdown

Thank you for your contribution to the Solidity compiler! A team member will follow up shortly.

If you haven't read our contributing guidelines and our review checklist before, please do it now, this makes the reviewing process and accepting your contribution smoother.

If you have any questions or need our help, feel free to post them in the PR or talk to us directly on the #solidity-dev channel on Matrix.

@CodeNinjaEvan CodeNinjaEvan merged commit 5802411 into tronprotocol:release_0.8.28 Jun 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants