Skip to content

Conversation

@Vectorized
Copy link
Owner

Description

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Ran forge fmt?
  • Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

@github-actions
Copy link

github-actions bot commented Aug 25, 2025

Gas Snapshot Comparison Report

Generated at commit : f3ca6a5, Compared to commit : 33b4b98

Contract Name Test Name Main Gas PR Gas Diff
LibBytesTest testBytesInCalldata() 2764 2742 -22
testCmp() 2607 2630 23
testDirectReturn() 9283 9342 59
testDynamicStructInCalldata() 4517 4473 -44
testIndexOfByte() 5434 5477 43
testSliceCalldata() 3131 3154 23
testStaticStructInCalldata() 20760 20628 -132
test__codesize() 14506 15779 1273
testCheckInCalldata() - 343 -
SafeTransferLibTest testApproveWithMissingReturn() 32677 32611 -66
testApproveWithRetryWithNonContractReverts() 7312 7334 22
testApproveWithReturnsFalseReverts() 10252 10274 22
testApproveWithReturnsTooMuch() 32913 32935 22
testApproveWithRevertingReverts() 10180 10202 22
testBalanceOfStandardERC20() 7842 7864 22
testPermit2() 90457 90479 22
testPermit2AnythingWithNonContractReverts() 125192 125147 -45
testPermit2OnDAI() 111021 111043 22
testSimplePermit2AndPermit2TransferFrom() 1120633 1120875 242
testSimplePermit2AndPermit2TransferFromGas() 139030 139052 22
testTotalSupplyQuery() 27469 27271 -198
testTransferAllETHToContractWithoutFallbackReverts() 10837 10859 22
testTransferAllFromWithStandardERC20() 33929 33862 -67
testTransferETHToContractWithoutFallbackReverts() 10872 10894 22
testTransferFromWithMissingReturn() 537973 538053 80
testTransferFromWithReturnsFalseReverts() 504634 504255 -379
testTransferFromWithReturnsTooLittleReverts() 504387 504310 -77
testTransferFromWithReturnsTooMuch() 538770 538712 -58
testTransferFromWithRevertingReverts() 548104 497115 -50989
testTransferFromWithStandardERC20() 537558 537499 -59
testTransferWithMissingReturn() 524785 524708 -77
testTransferWithReturnsFalseReverts() 501418 501359 -59
testTransferWithReturnsTooLittleReverts() 501292 501309 17
testTransferWithReturnsTooMuch() 525264 525205 -59
testTransferWithRevertingReverts() 501359 501318 -41
testTransferWithStandardERC20() 525368 525132 -236
test__codesize() 76553 76547 -6

@Vectorized Vectorized merged commit 43f04e8 into main Aug 25, 2025
11 checks passed
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