Skip to content

perf(es/minifier): Do heavy operation only if required#9902

Merged
kdy1 merged 2 commits intoswc-project:mainfrom
kdy1:par-usage
Jan 19, 2025
Merged

perf(es/minifier): Do heavy operation only if required#9902
kdy1 merged 2 commits intoswc-project:mainfrom
kdy1:par-usage

Conversation

@kdy1
Copy link
Copy Markdown
Member

@kdy1 kdy1 commented Jan 19, 2025

Description:

In store_var_for_inlining, we had called inline_with_multi_replacer needlessly. This PR fixes it.

@kdy1 kdy1 added this to the Planned milestone Jan 19, 2025
@kdy1 kdy1 requested a review from a team as a code owner January 19, 2025 17:55
kodiakhq[bot]
kodiakhq Bot previously approved these changes Jan 19, 2025
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 19, 2025

🦋 Changeset detected

Latest commit: ebec046

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

Comment thread crates/swc_ecma_minifier/src/compress/optimize/inline.rs
@kdy1 kdy1 self-assigned this Jan 19, 2025
@kdy1 kdy1 requested a review from a team as a code owner January 19, 2025 22:58
@kdy1 kdy1 merged commit 2687231 into swc-project:main Jan 19, 2025
@kdy1 kdy1 deleted the par-usage branch January 19, 2025 22:58
kdy1 added a commit to vercel/next.js that referenced this pull request Jan 20, 2025
### What?

Update SWC crates.

- ChangeLog: swc-project/swc@swc_core@v10.3.0...swc_core@v10.5.0


### Why?

To keep in sync and apply minifier perf PRs like

 - swc-project/swc#9894
 - swc-project/swc#9895
 - swc-project/swc#9900
 - swc-project/swc#9901
 - swc-project/swc#9902


### How?
@kdy1 kdy1 modified the milestones: Planned, v1.10.9 Jan 21, 2025
@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