perf(es/minifier): Improve arrow function inlining cost analysis

[kdy1]

Description:

This PR fixes the cost analysis of arrow functions by making it use Option<u8> instead of bool.

[changeset-bot]

⚠️ No Changeset found

Latest commit: eac8344

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[kdy1]

• perf(es/minifier): Improve arrow function inlining cost analysis #10093 👈 (View in Graphite)
• fix(es/minifier): Inline before cost analysis #10092
• fix(es/minifier): Fix insufficient logging #10091
• fix(es/minifier): Fix cargo feature debug #10090
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

CodSpeed Performance Report

Merging #10093 will degrade performances by 4.17%

_{Comparing kdy1/minifier-consider-cost (7e020de) with main (1425b56)}

Summary

⚡ 2 improvements
❌ 1 regressions
✅ 188 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

	Benchmark	BASE	HEAD	Change
❌	es/full/minify/libraries/lodash	188.2 ms	196.4 ms	-4.17%
⚡	es/minifier/libs/d3	525.3 ms	509 ms	+3.2%
⚡	es/minifier/libs/lodash	158.2 ms	153.1 ms	+3.32%

[kdy1]

Merge activity

• Feb 26, 10:13 PM GMT+9: A user merged this pull request with Graphite.