Skip to content

perf(es/minifier): Invert cache to be really a cache#9910

Merged
kdy1 merged 2 commits intoswc-project:mainfrom
kdy1:min-3
Jan 21, 2025
Merged

perf(es/minifier): Invert cache to be really a cache#9910
kdy1 merged 2 commits intoswc-project:mainfrom
kdy1:min-3

Conversation

@kdy1
Copy link
Copy Markdown
Member

@kdy1 kdy1 commented Jan 21, 2025

Description:

Previous PR may decrease the performance if the result of collect_infects_to is empty.

Related issue:

@kdy1 kdy1 added this to the Planned milestone Jan 21, 2025
@kdy1 kdy1 self-assigned this Jan 21, 2025
@kdy1 kdy1 requested a review from a team as a code owner January 21, 2025 11:26
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 21, 2025

🦋 Changeset detected

Latest commit: 68228d9

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

kodiakhq[bot]
kodiakhq Bot previously approved these changes Jan 21, 2025
@kdy1 kdy1 enabled auto-merge (squash) January 21, 2025 11:26
@kdy1 kdy1 requested a review from a team as a code owner January 21, 2025 11:26
@kdy1 kdy1 disabled auto-merge January 21, 2025 11:29
@kdy1 kdy1 merged commit 8bfb0e5 into swc-project:main Jan 21, 2025
@kdy1 kdy1 deleted the min-3 branch January 21, 2025 11:29
@kdy1 kdy1 modified the milestones: Planned, v1.10.9 Jan 21, 2025
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jan 21, 2025

CodSpeed Performance Report

Merging #9910 will improve performances by 15.4%

Comparing kdy1:min-3 (9538541) with main (4a3be8d)

Summary

⚡ 1 improvements
✅ 195 untouched benchmarks

Benchmarks breakdown

Benchmark main kdy1:min-3 Change
es/full/base/fixer 137.8 µs 119.4 µs +15.4%

@swc-project swc-project locked as resolved and limited conversation to collaborators Feb 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant