VTVL contest - WilliamAmbrozic's results

Gas Optimizations

22/09/2022 14:57 VTVL

Savings

The following changes pass the given tests saving approximately 47 BPS (optimizer off).

From

To

Tools Used: Manual Analysis, the given test suite.

VTVLVesting.sol

There is no need to initialize variables with default values

• Line: 27
• Line 148
• Line 353

Reference

Inline variables to save SLOAD

• Line 124
• Line 197

Example:

function vestedAmount(address _recipient, uint40 _referenceTs) public view returns (uint112) {
    Claim storage _claim = claims[_recipient];
    return _baseVestedAmount(_claim, _referenceTs);
}

changed to

function vestedAmount(address _recipient, uint40 _referenceTs) public view returns (uint112) {
    return _baseVestedAmount(claims[_recipient], _referenceTs);
}

hasNoClaim modifier is used once (can be inlined)

• Lines 123, 253

++i is cheaper than i++

• Line 353

Suggestion: change for (uint256 i = 0; i < length; i++) to for (uint256 i; i < length; ++i)

Cache variables to save an SLOAD

• usrClaim.amountWithdrawn (374-377)
• _claim.amountWithdrawn (426-429)