Skip to content

Commit

Permalink
Fix ling and gas checks
Browse files Browse the repository at this point in the history
  • Loading branch information
forshtat committed Oct 30, 2023
1 parent b8c730f commit b49fafe
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 11 deletions.
1 change: 1 addition & 0 deletions contracts/test/TestPaymasterRevertCustomError.sol
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ contract TestPaymasterRevertCustomError is BasePaymaster {
revert CustomError();
}
else if (revertType == RevertType.entryPointError){
// solium-disable-next-line security/no-inline-assembly
assembly {
mstore(0, INNER_OUT_OF_GAS)
revert(0, 32)
Expand Down
22 changes: 11 additions & 11 deletions reports/gas-checker.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,36 +4,36 @@
╔══════════════════════════╤════════╗
║ gas estimate "simple" │ 29014 ║
╟──────────────────────────┼────────╢
║ gas estimate "big tx 5k" │ 125260
║ gas estimate "big tx 5k" │ 125248
╚══════════════════════════╧════════╝

╔════════════════════════════════╤═══════╤═══════════════╤════════════════╤═════════════════════╗
║ handleOps description │ count │ total gasUsed │ per UserOp gas │ per UserOp overhead ║
║ │ │ │ (delta for │ (compared to ║
║ │ │ │ one UserOp) │ account.exec()) ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple │ 1 │ 81945 │ │ ║
║ simple │ 1 │ 81957 │ │ ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple - diff from previous │ 2 │ │ 4422615212
║ simple - diff from previous │ 2 │ │ 4419015176
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple │ 10 │ 480000 │ │ ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple - diff from previous │ 11 │ │ 44238 │ 15224 ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple paymaster │ 1 │ 88255 │ │ ║
║ simple paymaster │ 1 │ 88243 │ │ ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple paymaster with diff │ 2 │ │ 4320614192
║ simple paymaster with diff │ 2 │ │ 4319414180
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple paymaster │ 10 │ 477410 │ │ ║
║ simple paymaster │ 10 │ 477398 │ │ ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ simple paymaster with diff │ 11 │ │ 4324114227
║ simple paymaster with diff │ 11 │ │ 4326514251
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ big tx 5k │ 1 │ 183014 │ │ ║
║ big tx 5k │ 1 │ 182978 │ │ ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ big tx - diff from previous │ 2 │ │ 144725 │ 19465 ║
║ big tx - diff from previous │ 2 │ │ 144713 │ 19465 ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ big tx 5k │ 10 │ 1485632 │ │ ║
║ big tx 5k │ 10 │ 1485596 │ │ ║
╟────────────────────────────────┼───────┼───────────────┼────────────────┼─────────────────────╢
║ big tx - diff from previous │ 11 │ │ 14478619526
║ big tx - diff from previous │ 11 │ │ 14475019502
╚════════════════════════════════╧═══════╧═══════════════╧════════════════╧═════════════════════╝

0 comments on commit b49fafe

Please sign in to comment.