03-18-feat_minifier_support_keep_names_option - Branch - oxc-project/oxc

feat(minifier): support `keep_names` option

#9867Merged

Comparing

03-18-feat_minifier_support_keep_names_option

(

dcd356e

) with

main

(

b9cf61e

)

Untouched: 33

Benchmarks

Passed

lexer[RadixUIAdoptionSection.jsx]

tasks/benchmark/benches/lexer.rs::lexer::bench_lexer

21.1 µs21 µs

semantic[RadixUIAdoptionSection.jsx]

tasks/benchmark/benches/semantic.rs::semantic::bench_semantic

74.3 µs74.2 µs

mangler[typescript.js]

tasks/benchmark/benches/minifier.rs::minifier::bench_mangler

39.4 ms39.3 ms

semantic[pdf.mjs]

tasks/benchmark/benches/semantic.rs::semantic::bench_semantic

17.1 ms17.1 ms

parser[RadixUIAdoptionSection.jsx]

tasks/benchmark/benches/parser.rs::parser::bench_parser

91.3 µs91.2 µs

transformer[pdf.mjs]

tasks/benchmark/benches/transformer.rs::transformer::bench_transformer

10.3 ms10.3 ms

transformer[antd.js]

tasks/benchmark/benches/transformer.rs::transformer::bench_transformer

51.5 ms51.5 ms

minifier[react.development.js]

tasks/benchmark/benches/minifier.rs::minifier::bench_minifier

1.8 ms1.8 ms

semantic[checker.ts]

tasks/benchmark/benches/semantic.rs::semantic::bench_semantic

64.4 ms64.4 ms

mangler[antd.js]

tasks/benchmark/benches/minifier.rs::minifier::bench_mangler

15.8 ms15.8 ms

parser[pdf.mjs]

tasks/benchmark/benches/parser.rs::parser::bench_parser

18.1 ms18.1 ms

codegen_sourcemap[checker.ts]

tasks/benchmark/benches/codegen.rs::codegen::bench_codegen

65.7 ms65.7 ms

transformer[checker.ts]

tasks/benchmark/benches/transformer.rs::transformer::bench_transformer

23.1 ms23.1 ms

minifier[antd.js]

tasks/benchmark/benches/minifier.rs::minifier::bench_minifier

163.2 ms163.2 ms

linter[cal.com.tsx]

tasks/benchmark/benches/linter.rs::linter::bench_linter

1.2 s1.2 s

parser[antd.js]

tasks/benchmark/benches/parser.rs::parser::bench_parser

112.6 ms112.6 ms

isolated-declarations[vue-id.ts]

tasks/benchmark/benches/isolated_declarations.rs::transformer::bench_isolated_declarations

57.8 ms57.8 ms

estree[checker.ts]

tasks/benchmark/benches/parser.rs::parser::bench_estree

87.5 ms87.5 ms

codegen[checker.ts]

tasks/benchmark/benches/codegen.rs::codegen::bench_codegen

22.6 ms22.6 ms

parser[checker.ts]

tasks/benchmark/benches/parser.rs::parser::bench_parser

56.4 ms56.4 ms

lexer[checker.ts]

tasks/benchmark/benches/lexer.rs::lexer::bench_lexer

14.5 ms14.5 ms

lexer[antd.js]

tasks/benchmark/benches/lexer.rs::lexer::bench_lexer

24.1 ms24.1 ms

lexer[pdf.mjs]

tasks/benchmark/benches/lexer.rs::lexer::bench_lexer

3.8 ms3.8 ms

lexer[cal.com.tsx]

tasks/benchmark/benches/lexer.rs::lexer::bench_lexer

5.7 ms5.7 ms

linter[checker.ts]

tasks/benchmark/benches/linter.rs::linter::bench_linter

3 s3 s

minifier[typescript.js]

tasks/benchmark/benches/minifier.rs::minifier::bench_minifier

288.2 ms288.2 ms

semantic[antd.js]

tasks/benchmark/benches/semantic.rs::semantic::bench_semantic

104.9 ms104.9 ms

parser[cal.com.tsx]

tasks/benchmark/benches/parser.rs::parser::bench_parser

29.8 ms29.8 ms

linter[RadixUIAdoptionSection.jsx]

tasks/benchmark/benches/linter.rs::linter::bench_linter

2.7 ms2.7 ms

semantic[cal.com.tsx]

tasks/benchmark/benches/semantic.rs::semantic::bench_semantic

26 ms26 ms

mangler[react.development.js]

tasks/benchmark/benches/minifier.rs::minifier::bench_mangler

292.9 µs293.3 µs

transformer[cal.com.tsx]

tasks/benchmark/benches/transformer.rs::transformer::bench_transformer

-1%

34.3 ms34.6 ms

transformer[RadixUIAdoptionSection.jsx]

tasks/benchmark/benches/transformer.rs::transformer::bench_transformer

-1%

144.5 µs146 µs

Commits

Click on a commit to change the comparison range

Base

main

b9cf61e

-0.04%

feat(minifier): support `keep_names` option (#9867) Added support for `keep_names: { function: true, class: true }`. Setting `.name` is makes it difficult to treeshake later on (https://github.com/evanw/esbuild/issues/3965), so it is better to avoid if possible. This PR only adds support for the compressor. I'll add support for the mangler later on (probably first add required information to the semantics). refs #9711

dcd356e

10 months ago

by sapphi-red

Home Terms Privacy Docs