Skip to content

Commit

Permalink
Enable via IR compilation / memory safe issue fix (#692)
Browse files Browse the repository at this point in the history
* annotate functions as memory-safe to pass compilation via ir

* Adjust solidity version and increase optimizer runs

* Remove gas tests that fail because via IR optimizes the code in a way that the gas overhead subtraction in the gas snapshot library causes an underflow

* regenerate gas snapshots

* regenerate gas snapshots

* increase optimizer runs
  • Loading branch information
gretzke authored May 28, 2024
1 parent 6e6ce35 commit 35edd8e
Show file tree
Hide file tree
Showing 103 changed files with 102 additions and 121 deletions.
2 changes: 1 addition & 1 deletion .forge-snapshots/BitMathLeastSignificantBitMaxUint128.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
455
29
2 changes: 1 addition & 1 deletion .forge-snapshots/BitMathLeastSignificantBitMaxUint256.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
457
29
Original file line number Diff line number Diff line change
@@ -1 +1 @@
453
29
2 changes: 1 addition & 1 deletion .forge-snapshots/BitMathMostSignificantBitMaxUint128.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
393
29
2 changes: 1 addition & 1 deletion .forge-snapshots/BitMathMostSignificantBitMaxUint256.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
413
29
2 changes: 1 addition & 1 deletion .forge-snapshots/BitMathMostSignificantBitSmallNumber.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
321
29
2 changes: 1 addition & 1 deletion .forge-snapshots/ERC6909Claims approve.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
46492
46323
2 changes: 1 addition & 1 deletion .forge-snapshots/ERC6909Claims burn.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
29428
29389
2 changes: 1 addition & 1 deletion .forge-snapshots/ERC6909Claims mint.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
46815
46603
2 changes: 1 addition & 1 deletion .forge-snapshots/ERC6909Claims transfer.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
52046
51756
Original file line number Diff line number Diff line change
@@ -1 +1 @@
54699
54437
Original file line number Diff line number Diff line change
@@ -1 +1 @@
60247
59939
Original file line number Diff line number Diff line change
@@ -1 +1 @@
57049
56770
1 change: 0 additions & 1 deletion .forge-snapshots/HooksShouldCallBeforeSwap.snap

This file was deleted.

2 changes: 1 addition & 1 deletion .forge-snapshots/NoDelegateCall.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
41
51
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_oneForZero_exactInCapped.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1662
1157
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_oneForZero_exactInPartial.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2152
1392
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_oneForZero_exactOutCapped.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1413
951
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_oneForZero_exactOutPartial.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2120
1506
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_zeroForOne_exactInCapped.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1753
1118
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_zeroForOne_exactInPartial.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2671
1832
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_zeroForOne_exactOutCapped.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1504
914
2 changes: 1 addition & 1 deletion .forge-snapshots/SwapMath_zeroForOne_exactOutPartial.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1850
1119
2 changes: 1 addition & 1 deletion .forge-snapshots/TickMathGetSqrtPriceAtTick.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
74032
70953
2 changes: 1 addition & 1 deletion .forge-snapshots/TickMathGetTickAtSqrtPrice.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
216101
197704
Original file line number Diff line number Diff line change
@@ -1 +1 @@
149323
146557
2 changes: 1 addition & 1 deletion .forge-snapshots/addLiquidity CA fee.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
325681
320738
2 changes: 1 addition & 1 deletion .forge-snapshots/addLiquidity with empty hook.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
279677
275714
2 changes: 1 addition & 1 deletion .forge-snapshots/addLiquidity with native token.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
139505
136837
1 change: 0 additions & 1 deletion .forge-snapshots/calculateSwapFee.snap

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1 +1 @@
297501
294762
2 changes: 1 addition & 1 deletion .forge-snapshots/donate gas with 1 token.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
106609
106027
2 changes: 1 addition & 1 deletion .forge-snapshots/donate gas with 2 tokens.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
147104
146035
2 changes: 1 addition & 1 deletion .forge-snapshots/erc20 collect protocol fees.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
57229
57224
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getFeeGrowthGlobals.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
661
704
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getFeeGrowthInside.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
415
375
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getLiquidity.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
415
375
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getPositionInfo.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
817
874
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getPositionLiquidity.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
415
375
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getSlot0.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
415
375
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getTickBitmap.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
415
375
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getTickFeeGrowthOutside.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
661
704
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getTickInfo.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
817
874
2 changes: 1 addition & 1 deletion .forge-snapshots/extsload getTickLiquidity.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
415
375
Original file line number Diff line number Diff line change
@@ -1 +1 @@
5244
5113
Original file line number Diff line number Diff line change
@@ -1 +1 @@
22341
22213
Original file line number Diff line number Diff line change
@@ -1 +1 @@
5350
5174
Original file line number Diff line number Diff line change
@@ -1 +1 @@
502
247
Original file line number Diff line number Diff line change
@@ -1 +1 @@
638
364
Original file line number Diff line number Diff line change
@@ -1 +1 @@
369
198
Original file line number Diff line number Diff line change
@@ -1 +1 @@
370
198
Original file line number Diff line number Diff line change
@@ -1 +1 @@
520
320
Original file line number Diff line number Diff line change
@@ -1 +1 @@
711
543
Original file line number Diff line number Diff line change
@@ -1 +1 @@
746
542
Original file line number Diff line number Diff line change
@@ -1 +1 @@
398
209
2 changes: 1 addition & 1 deletion .forge-snapshots/getReserves.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1167
1630
2 changes: 1 addition & 1 deletion .forge-snapshots/initialize.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
61173
59576
2 changes: 1 addition & 1 deletion .forge-snapshots/native collect protocol fees.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
59496
59588
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2488
2270
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2488
2278
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2488
2285
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2475
2262
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2784
2615
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2475
2291
2 changes: 1 addition & 1 deletion .forge-snapshots/poolManager bytecode size.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
19769
23805
2 changes: 1 addition & 1 deletion .forge-snapshots/removeLiquidity CA fee.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
181200
176325
2 changes: 1 addition & 1 deletion .forge-snapshots/removeLiquidity with empty hook.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
135371
131597
2 changes: 1 addition & 1 deletion .forge-snapshots/removeLiquidity with native token.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
115664
113398
2 changes: 1 addition & 1 deletion .forge-snapshots/set protocol fee.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
32478
31702
Original file line number Diff line number Diff line change
@@ -1 +1 @@
102765
98747
2 changes: 1 addition & 1 deletion .forge-snapshots/simple addLiquidity.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
165257
161227
Original file line number Diff line number Diff line change
@@ -1 +1 @@
96455
92503
2 changes: 1 addition & 1 deletion .forge-snapshots/simple removeLiquidity.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
88495
84560
2 changes: 1 addition & 1 deletion .forge-snapshots/simple swap with native.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
114299
109712
2 changes: 1 addition & 1 deletion .forge-snapshots/simple swap.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
129464
124538
2 changes: 1 addition & 1 deletion .forge-snapshots/sparse external sload.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2007
2057
2 changes: 1 addition & 1 deletion .forge-snapshots/swap CA custom curve + swap noop.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
131734
128990
2 changes: 1 addition & 1 deletion .forge-snapshots/swap CA fee on unspecified.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
178608
174512
Original file line number Diff line number Diff line change
@@ -1 +1 @@
109936
108127
2 changes: 1 addition & 1 deletion .forge-snapshots/swap against liquidity.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
121307
119479
Loading

0 comments on commit 35edd8e

Please sign in to comment.