perf(es/minifier): Merge dead_branch_remover into pure optimizer

[kdy1]

Description:

Merge dead_branch_remover into the pure optimizer to make sequential process parallel and reduce the overhead of the visitor pattern.

Related issue:

• Closes Revisit all logic in dead_branch_remover #10217 because we don't use it for minifier anymore.

[changeset-bot]

🦋 Changeset detected

Latest commit: b02ff3d

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

[kdy1]

• perf(es/minifier): Merge dead_branch_remover into pure optimizer #10201 : 2 dependent PRs (#10199 , #10202 ) 👈 (View in Graphite)
• main

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

[codspeed-hq]

CodSpeed Performance Report

Merging #10201 will improve performances by 2.18%

_{Comparing kdy1/dead-code (b02ff3d) with main (b192dc8)}

Summary

⚡ 8 improvements
✅ 144 untouched benchmarks

Benchmarks breakdown

	Benchmark	BASE	HEAD	Change
⚡	serialization of serde	2.8 µs	2.8 µs	+1.06%
⚡	es/minifier/libs/antd	2.7 s	2.6 s	+2.04%
⚡	es/minifier/libs/echarts	1.9 s	1.8 s	+2.18%
⚡	es/minifier/libs/jquery	122.3 ms	121 ms	+1.02%
⚡	es/minifier/libs/lodash	142.3 ms	140.9 ms	+1.01%
⚡	es/minifier/libs/three	779.2 ms	764 ms	+1.98%
⚡	es/minifier/libs/typescript	4.6 s	4.5 s	+2.16%
⚡	es/minifier/libs/victory	1.1 s	1.1 s	+2.18%

[Copilot]

Pull Request Overview

This PR integrates the functionality of dead_branch_remover into the pure optimizer to enable parallel processing and reduce visitor pattern overhead. Key changes include updating test snapshot sizes, adjusting control flow constructs in various minified test files, and refactoring duration formatting in the output index file.

Reviewed Changes

Copilot reviewed 104 out of 105 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
crates/swc/tests/libs-size.snapshot.md	Minor numeric adjustments in file sizes for several JS files
.changeset/wild-berries-divide.md	Adds a changeset to document version bumps and the perf related change
crates/swc/tests/tsc-references/*.minified.js	Updates control flow labels and statement formatting in test references
crates/swc/tests/vercel/full/ms/1/output/index.ts	Refactors helper function name and error handling in duration formatting
crates/swc/tests/tsc-references/parserindenter.2.minified.js	Reformats nested if blocks for indentation handling
crates/swc/tests/tsc-references/assignmentCompatWithObjectMembersAccessibility.2.minified.js	Updates chain assignment order for object members

Files not reviewed (1)

• crates/swc_ecma_minifier/scripts/test.sh: Language not supported

Comments suppressed due to low confidence (2)

crates/swc/tests/tsc-references/assignmentCompatWithObjectMembersAccessibility.2.minified.js:4

• The chain assignment for the second block appears truncated compared to the original version. Verify if the removal of assignments for e1 is intentional to maintain the intended behavior.

TargetIsPublic || (TargetIsPublic = {}), a = b, a = i, a = d, b = a = e, b = i, b = d, b = e, i = a, i = b, i = d, i = e, d = a, d = b, d = i, d = e, e = a, e = b, e = i, e = d, TargetIsPublic || (TargetIsPublic = {}), a1 = b1, a1 = i1, a1 = d1, b1 = a1 = e1, b1 = i1, b1 = d1, b1 = e1, i1 = a1, i1 = b1, i1 = d1, i1 = e1, d1 = a1, d1 = b1, d1 = i1;

crates/swc/tests/tsc-references/parser_continueTarget3.2.minified.js:2

• Replacing 'for(;;)continue;' with 'for(;;);' removes the continue statement, which may alter the loop’s behavior. Please verify that this change is intentional.

for(;;);

[kdy1]

Requires merging expr_simplifier into the pure minifier

[kdy1]

@swc-project/core-es Can you review only rust code? I'm checking test diffs

[kdy1]

I'll merge this, but feel free to left a review as I can invert PR afterwards.