search

KyberNetwork / smart-contracts

Main smart contracts for Kyber Network, including the main platform contract, reserve contracts etc.
https://kyber.network
MIT License
378 stars 340 forks source link

[don't pull] Compare gas used when validating if reserve is listed with specific token #970

Closed ducquangkstn closed 4 years ago

ducquangkstn commented 4 years ago

gas Report

┌─────────────────────────────────────────────────────────┬──────────┬─────────┬───────┐
│                         (index)                         │ Katalyst │ current │ diff  │
├─────────────────────────────────────────────────────────┼──────────┼─────────┼───────┤
│      t2e trade, type: SPLIT, fee: 123, reserves: 3      │  576089  │ 579741  │ 3652  │
│      e2t trade, type: SPLIT, fee: 123, reserves: 3      │  561120  │ 564739  │ 3619  │
│      t2t trade, type: SPLIT, fee: 123, reserves: 3      │  979785  │ 987045  │ 7260  │
│     t2e trade, type: NO HINT, fee: 123, reserves: 3     │  501635  │ 507444  │ 5809  │
│     e2t trade, type: NO HINT, fee: 123, reserves: 3     │  540672  │ 546448  │ 5776  │
│     t2t trade, type: NO HINT, fee: 123, reserves: 3     │  918461  │ 930035  │ 11574 │
│     t2e trade, type: MASK_IN, fee: 123, reserves: 3     │  393024  │ 396652  │ 3628  │
│     e2t trade, type: MASK_IN, fee: 123, reserves: 3     │  416721  │ 420316  │ 3595  │
│     t2t trade, type: MASK_IN, fee: 123, reserves: 3     │  691987  │ 699223  │ 7236  │
│    t2e trade, type: MASK_OUT, fee: 123, reserves: 2     │  404575  │ 408215  │ 3640  │
│    e2t trade, type: MASK_OUT, fee: 123, reserves: 2     │  427702  │ 431330  │ 3628  │
│    t2t trade, type: MASK_OUT, fee: 123, reserves: 2     │  715216  │ 722485  │ 7269  │
│  t2e trade maxdest, type: SPLIT, fee: 123, reserves: 3  │  561145  │ 564797  │ 3652  │
│  e2t trade maxdest, type: SPLIT, fee: 123, reserves: 3  │  575303  │ 578922  │ 3619  │
│  t2t trade maxdest, type: SPLIT, fee: 123, reserves: 3  │  999065  │ 1006324 │ 7259  │
│ t2e trade maxdest, type: NO HINT, fee: 123, reserves: 3 │  513124  │ 518933  │ 5809  │
│ e2t trade maxdest, type: NO HINT, fee: 123, reserves: 3 │  551079  │ 556855  │ 5776  │
│ t2t trade maxdest, type: NO HINT, fee: 123, reserves: 3 │  950509  │ 962082  │ 11573 │
└─────────────────────────────────────────────────────────┴──────────┴─────────┴───────┘