use snapLastCall and isolate for posm benchmarks

Uniswap · Jun 13, 2024 · da91136 · da91136
1 parent c8ce67b
commit da91136
Show file tree

Hide file tree

Showing 20 changed files with 25 additions and 30 deletions.
diff --git a/.forge-snapshots/FullRangeAddInitialLiquidity.snap b/.forge-snapshots/FullRangeAddInitialLiquidity.snap
@@ -1 +1 @@
-311137
+354433
diff --git a/.forge-snapshots/FullRangeAddLiquidity.snap b/.forge-snapshots/FullRangeAddLiquidity.snap
@@ -1 +1 @@
-122946
+161742
diff --git a/.forge-snapshots/FullRangeFirstSwap.snap b/.forge-snapshots/FullRangeFirstSwap.snap
@@ -1 +1 @@
-80287
+146467
diff --git a/.forge-snapshots/FullRangeInitialize.snap b/.forge-snapshots/FullRangeInitialize.snap
@@ -1 +1 @@
-1015181
+1037821
diff --git a/.forge-snapshots/FullRangeRemoveLiquidity.snap b/.forge-snapshots/FullRangeRemoveLiquidity.snap
@@ -1 +1 @@
-110544
+146372
diff --git a/.forge-snapshots/FullRangeRemoveLiquidityAndRebalance.snap b/.forge-snapshots/FullRangeRemoveLiquidityAndRebalance.snap
@@ -1 +1 @@
-240022
+281650
diff --git a/.forge-snapshots/FullRangeSecondSwap.snap b/.forge-snapshots/FullRangeSecondSwap.snap
@@ -1 +1 @@
-45997
+116177
diff --git a/.forge-snapshots/FullRangeSwap.snap b/.forge-snapshots/FullRangeSwap.snap
@@ -1 +1 @@
-79418
+145886
diff --git a/.forge-snapshots/OracleGrow10Slots.snap b/.forge-snapshots/OracleGrow10Slots.snap
@@ -1 +1 @@
-232960
+254164
diff --git a/.forge-snapshots/OracleGrow10SlotsCardinalityGreater.snap b/.forge-snapshots/OracleGrow10SlotsCardinalityGreater.snap
@@ -1 +1 @@
-223649
+249653
diff --git a/.forge-snapshots/OracleGrow1Slot.snap b/.forge-snapshots/OracleGrow1Slot.snap
@@ -1 +1 @@
-32845
+54049
diff --git a/.forge-snapshots/OracleGrow1SlotCardinalityGreater.snap b/.forge-snapshots/OracleGrow1SlotCardinalityGreater.snap
@@ -1 +1 @@
-23545
+49549
diff --git a/.forge-snapshots/OracleInitialize.snap b/.forge-snapshots/OracleInitialize.snap
@@ -1 +1 @@
-51310
+72794
diff --git a/.forge-snapshots/TWAMMSubmitOrder.snap b/.forge-snapshots/TWAMMSubmitOrder.snap
@@ -1 +1 @@
-122359
+156851
diff --git a/.forge-snapshots/decreaseLiquidity_erc20.snap b/.forge-snapshots/decreaseLiquidity_erc20.snap
@@ -1 +1 @@
-114609
+187091
diff --git a/.forge-snapshots/decreaseLiquidity_erc6909.snap b/.forge-snapshots/decreaseLiquidity_erc6909.snap
@@ -1 +1 @@
-112540
+166084
diff --git a/.forge-snapshots/increaseLiquidity_erc20.snap b/.forge-snapshots/increaseLiquidity_erc20.snap
@@ -1 +1 @@
-74128
+187781
diff --git a/.forge-snapshots/increaseLiquidity_erc6909.snap b/.forge-snapshots/increaseLiquidity_erc6909.snap
@@ -1 +1 @@
-77920
+163384
diff --git a/.forge-snapshots/mintWithLiquidity.snap b/.forge-snapshots/mintWithLiquidity.snap
@@ -1 +1 @@
-475866
+485624
diff --git a/test/position-managers/Gas.t.sol b/test/position-managers/Gas.t.sol
@@ -80,45 +80,40 @@ contract GasTest is Test, Deployers, GasSnapshot {
     //     });
     //     snapStart("mint");
     //     lpm.mint(params);
-    //     snapEnd();
+    //     snapLastCall();
     // }
 
     function test_gas_mintWithLiquidity() public {
-        snapStart("mintWithLiquidity");
         lpm.mint(range, 10_000 ether, block.timestamp + 1, address(this), ZERO_BYTES);
-        snapEnd();
+        snapLastCall("mintWithLiquidity");
     }
 
     function test_gas_increaseLiquidity_erc20() public {
         (uint256 tokenId,) = lpm.mint(range, 10_000 ether, block.timestamp + 1, address(this), ZERO_BYTES);
 
-        snapStart("increaseLiquidity_erc20");
         lpm.increaseLiquidity(tokenId, 1000 ether, ZERO_BYTES, false);
-        snapEnd();
+        snapLastCall("increaseLiquidity_erc20");
     }
 
     function test_gas_increaseLiquidity_erc6909() public {
         (uint256 tokenId,) = lpm.mint(range, 10_000 ether, block.timestamp + 1, address(this), ZERO_BYTES);
 
-        snapStart("increaseLiquidity_erc6909");
         lpm.increaseLiquidity(tokenId, 1000 ether, ZERO_BYTES, true);
-        snapEnd();
+        snapLastCall("increaseLiquidity_erc6909");
     }
 
     function test_gas_decreaseLiquidity_erc20() public {
         (uint256 tokenId,) = lpm.mint(range, 10_000 ether, block.timestamp + 1, address(this), ZERO_BYTES);
 
-        snapStart("decreaseLiquidity_erc20");
         lpm.decreaseLiquidity(tokenId, 10_000 ether, ZERO_BYTES, false);
-        snapEnd();
+        snapLastCall("decreaseLiquidity_erc20");
     }
 
     function test_gas_decreaseLiquidity_erc6909() public {
         (uint256 tokenId,) = lpm.mint(range, 10_000 ether, block.timestamp + 1, address(this), ZERO_BYTES);
 
-        snapStart("decreaseLiquidity_erc6909");
         lpm.decreaseLiquidity(tokenId, 10_000 ether, ZERO_BYTES, true);
-        snapEnd();
+        snapLastCall("decreaseLiquidity_erc6909");
     }
 
     function test_gas_burn() public {}