Skip to content

Commit 405cbb9

Browse files
committed
evm: Update sizes for per-chain
1 parent 9c3fc7e commit 405cbb9

File tree

1 file changed

+27
-12
lines changed

1 file changed

+27
-12
lines changed

evm/NOTES.md

Lines changed: 27 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ This integration branch is PR #21.
99
Note that the indentation indicates branch dependencies.
1010

1111
- **evm_TransceiverRegistry_split** (PR #22) - This branch splits transceiver admin into a separate contract.
12-
- **evm_per_chain_transceivers** (PR #26) - This branch adds per-chain transceiver support, should also add per-chain thresholds.
12+
- **evm_per_chain_transceivers** (PR #26) - This branch adds per-chain transceiver and per-chain threshold support.
1313
- **evm/add_MsgManager** (PR #23) - Creates `MsgManagerBase` and `MsgManager` and makes `NttManager` inherit from `MsgManagerBase`.
1414
- **evm/add_SharedWormholeTransceiver** (PR #25) - Adds a shareable transceiver.
1515

@@ -26,49 +26,65 @@ However, some of the other changes also cause that increase, so maybe we can rev
2626

2727
```bash
2828
evm (main)$ forge build --sizes --via-ir --skip test
29-
3029
╭-----------------------------------------+------------------+-------------------+--------------------+---------------------╮
3130
| Contract | Runtime Size (B) | Initcode Size (B) | Runtime Margin (B) | Initcode Margin (B) |
3231
+===========================================================================================================================+
3332
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
3433
| NttManager | 24,066 | 25,673 | 510 | 23,479 |
3534
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
3635
| NttManagerNoRateLimiting | 17,141 | 18,557 | 7,435 | 30,595 |
37-
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
38-
36+
╰-----------------------------------------+------------------+-------------------+--------------------+---------------------╯
3937
```
4038

4139
### After moving token and mode (the change that wasn't made)
4240

4341
```bash
4442
evm (main)$ forge build --sizes --via-ir --skip test
45-
43+
╭-----------------------------------------+------------------+-------------------+--------------------+---------------------╮
44+
| Contract | Runtime Size (B) | Initcode Size (B) | Runtime Margin (B) | Initcode Margin (B) |
45+
+===========================================================================================================================+
4646
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
4747
| NttManager | 24,066 | 25,676 | 510 | 23,476 |
4848
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
4949
| NttManagerNoRateLimiting | 18,788 | 20,281 | 5,788 | 28,871 |
50-
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
51-
50+
╰-----------------------------------------+------------------+-------------------+--------------------+---------------------╯
5251
```
5352

5453
## Creating TransceiverRegistryAdmin
5554

5655
```bash
5756
evm (main)$ forge build --sizes --via-ir --skip test
58-
57+
╭-----------------------------------------+------------------+-------------------+--------------------+---------------------╮
58+
| Contract | Runtime Size (B) | Initcode Size (B) | Runtime Margin (B) | Initcode Margin (B) |
59+
+===========================================================================================================================+
5960
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
6061
| NttManager | 23,220 | 26,937 | 1,356 | 22,215 |
6162
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
6263
| NttManagerNoRateLimiting | 16,254 | 19,713 | 8,322 | 29,439 |
63-
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
64+
╰-----------------------------------------+------------------+-------------------+--------------------+---------------------╯
65+
```
6466

67+
## Adding Per-Chain Transceivers (on top of TransceiverRegistryAdmin)
68+
69+
```bash
70+
evm (main)$ forge build --sizes --via-ir --skip test
71+
╭-----------------------------------------+------------------+-------------------+--------------------+---------------------╮
72+
| Contract | Runtime Size (B) | Initcode Size (B) | Runtime Margin (B) | Initcode Margin (B) |
73+
+===========================================================================================================================+
74+
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
75+
| NttManager | 24,089 | 31,480 | 487 | 17,672 |
76+
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
77+
| NttManagerNoRateLimiting | 17,150 | 24,275 | 7,426 | 24,877 |
78+
╰-----------------------------------------+------------------+-------------------+--------------------+---------------------╯
6579
```
6680

6781
## Creating MsgManagerBase
6882

6983
```bash
7084
evm (main)$ forge build --sizes --via-ir --skip test
71-
85+
╭-----------------------------------------+------------------+-------------------+--------------------+---------------------╮
86+
| Contract | Runtime Size (B) | Initcode Size (B) | Runtime Margin (B) | Initcode Margin (B) |
87+
+===========================================================================================================================+
7288
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
7389
| NttManager | 24,076 | 25,719 | 500 | 23,433 |
7490
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
@@ -77,6 +93,5 @@ evm (main)$ forge build --sizes --via-ir --skip test
7793
| MsgManager | 12,540 | 13,745 | 12,036 | 35,407 |
7894
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
7995
| MsgManagerWithExecutor | 13,145 | 14,400 | 11,431 | 34,752 |
80-
|-----------------------------------------+------------------+-------------------+--------------------+---------------------|
81-
96+
╰-----------------------------------------+------------------+-------------------+--------------------+---------------------╯
8297
```

0 commit comments

Comments
 (0)