Skip to content

Conversation

@aalavandhan
Copy link
Member

This reverts commit b150071.

Rolling back some of the gas optimization changes as the new version of the compiler/optimizer handles all of them.

·--------------------------------------------------|---------------------------|-------------|----------------------------·
|               Solc version: 0.7.6                ·  Optimizer enabled: true  ·  Runs: 200  ·  Block limit: 9500000 gas  │
···················································|···························|·············|·····························
|  Methods                                                                                                                │
·····················|·····························|·············|·············|·············|··············|··············
|  Contract          ·  Method                     ·  Min        ·  Max        ·  Avg        ·  # calls     ·  usd (avg)  │
·····················|·····························|·············|·············|·············|··············|··············
|  Orchestrator      ·  addTransaction             ·      71349  ·     131907  ·     109976  ·           6  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  Orchestrator      ·  rebase                     ·      27372  ·     123539  ·      80703  ·          80  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  Orchestrator      ·  removeTransaction          ·      28443  ·      51261  ·      37474  ·           3  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  Orchestrator      ·  setTransactionEnabled      ·      25842  ·      30030  ·      27936  ·           2  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  Orchestrator      ·  transferOwnership          ·          -  ·          -  ·      30900  ·           1  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  approve                    ·      16707  ·      46779  ·      36661  ·          34  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  decreaseAllowance          ·      28520  ·      32821  ·      30671  ·           8  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  increaseAllowance          ·      32813  ·      47837  ·      38825  ·          10  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  permit                     ·          -  ·          -  ·      76681  ·           3  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  rebase                     ·      28376  ·      41895  ·      38170  ·          13  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  setMonetaryPolicy          ·          -  ·          -  ·      47269  ·           8  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  transfer                   ·      39802  ·      54778  ·      53412  ·          11  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  transferAll                ·      31091  ·      39491  ·      37391  ·           4  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  transferAllFrom            ·      31049  ·      46049  ·      34527  ·           5  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragments        ·  transferFrom               ·          -  ·          -  ·      46473  ·           2  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragmentsPolicy  ·  setCpiOracle               ·      31235  ·      46235  ·      38735  ·           4  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragmentsPolicy  ·  setDeviationThreshold      ·      15066  ·      30126  ·      18078  ·           5  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragmentsPolicy  ·  setMarketOracle            ·      31234  ·      46234  ·      38734  ·           4  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragmentsPolicy  ·  setOrchestrator            ·      31189  ·      46177  ·      38683  ·           4  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragmentsPolicy  ·  setRebaseLag               ·          -  ·          -  ·      30190  ·           2  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  UFragmentsPolicy  ·  setRebaseTimingParameters  ·      25491  ·      40527  ·      29435  ·          15  ·          -  │
·····················|·····························|·············|·············|·············|··············|··············
|  Deployments                                     ·                                         ·  % of limit  ·             │
···················································|·············|·············|·············|··············|··············
|  Orchestrator                                    ·          -  ·          -  ·     836870  ·       8.8 %  ·          -  │
···················································|·············|·············|·············|··············|··············
|  UFragments                                      ·          -  ·          -  ·    1455717  ·      15.3 %  ·          -  │
···················································|·············|·············|·············|··············|··············
|  UFragmentsPolicy                                ·          -  ·          -  ·     947968  ·        10 %  ·          -  │
·--------------------------------------------------|-------------|-------------|-------------|--------------|-------------·

This reverts commit b150071.
@aalavandhan aalavandhan merged commit 212d341 into master Feb 16, 2021
@aalavandhan aalavandhan deleted the gas-rollback branch February 16, 2021 21:15
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.

4 participants