All optimizations were benchmarked using the protocol's tests using the following config: solc version 0.8.18, optimizer on, 200 runs, viaIR = true. In most cases forge test --gas-report was used
Each optimization was submitted individually.
Issue | Instances | Total Gas Saved | |
---|---|---|---|
[G‑01] | Consider using clones | * | -70% cheaper deployment gas |
[G‑02] | ReentrancyGuard can be safely removed | 1 | 42725 |
[G‑03] | computeEscrowAddress() can be internal instead of public | 1 | 55863 + 193 |
[G‑04] | Redundant zero address checks | 2 | 237 |
[G‑05] | Input validation should be done in the beginning | 2 | 110649(in the revert case) |
[G‑06] | Emit after the transfer has been made in case it fails | 2 | 1381(in the revert case) |
[G‑07] | The bytecode can be removed from the function params | 1 | 27 |
[G‑08] | Nested ifs are cheaper than && | 1 | 19 |
Instead of having computeEscrowAddress() accept the bytecode as an argument it would more efficient if it was already embedded in the keccak256 function. This will allow us to save a small amount of gas.
Gas Savings for newEscrow() obtained via protocol's tests(median was used because the avg isnt accurate here): 27 gas
MED | |
---|---|
Before | 627057 |
After | 627030 |
There is 1 instance of this issue:
Instead of passing in the byteCode into the function, it can be removed and you can put type(Escrow).creationCode in the abi.encodePacked() where its supposed to be
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.