mirror of
https://github.com/danbulant/oxc
synced 2026-05-19 04:08:41 +00:00
feat(minifier): imprve conditional expr minification with boolean lit (#8207)
This commit is contained in:
parent
3b4501183c
commit
3202b4fed0
2 changed files with 27 additions and 7 deletions
|
|
@ -319,7 +319,27 @@ impl<'a> PeepholeMinimizeConditions {
|
|||
}
|
||||
|
||||
// `x ? true : y` -> `x || y`
|
||||
// TODO
|
||||
// `x ? false : y` -> `!x && y`
|
||||
if let (Expression::Identifier(_), Expression::BooleanLiteral(consequent_lit), _) =
|
||||
(&expr.test, &expr.consequent, &expr.alternate)
|
||||
{
|
||||
if consequent_lit.value {
|
||||
let ident = ctx.ast.move_expression(&mut expr.test);
|
||||
return Some(ctx.ast.expression_logical(
|
||||
expr.span,
|
||||
ident,
|
||||
LogicalOperator::Or,
|
||||
ctx.ast.move_expression(&mut expr.alternate),
|
||||
));
|
||||
}
|
||||
let ident = ctx.ast.move_expression(&mut expr.test);
|
||||
return Some(ctx.ast.expression_logical(
|
||||
expr.span,
|
||||
ctx.ast.expression_unary(expr.span, UnaryOperator::LogicalNot, ident),
|
||||
LogicalOperator::And,
|
||||
ctx.ast.move_expression(&mut expr.alternate),
|
||||
));
|
||||
}
|
||||
|
||||
None
|
||||
}
|
||||
|
|
@ -624,8 +644,8 @@ mod test {
|
|||
fn test_minimize_expr_condition() {
|
||||
fold("(x ? true : false) && y()", "!!x && y()");
|
||||
fold("(x ? false : true) && y()", "!x && y()");
|
||||
// fold("(x ? true : y) && y()", "(x || y)&&y()");
|
||||
// fold("(x ? y : false) && y()", "(x && y)&&y()");
|
||||
fold("(x ? true : y) && y()", "(x || y) && y()");
|
||||
// fold("(x ? y : false) && y()", "(x && y) && y()");
|
||||
// fold("(x && true) && y()", "x && y()");
|
||||
// fold("(x && false) && y()", "0&&y()");
|
||||
// fold("(x || true) && y()", "1&&y()");
|
||||
|
|
|
|||
|
|
@ -13,15 +13,15 @@ Original | minified | minified | gzip | gzip | Fixture
|
|||
|
||||
555.77 kB | 273.48 kB | 270.13 kB | 90.94 kB | 90.80 kB | d3.js
|
||||
|
||||
1.01 MB | 460.75 kB | 458.89 kB | 126.88 kB | 126.71 kB | bundle.min.js
|
||||
1.01 MB | 460.75 kB | 458.89 kB | 126.87 kB | 126.71 kB | bundle.min.js
|
||||
|
||||
1.25 MB | 653.17 kB | 646.76 kB | 163.57 kB | 163.73 kB | three.js
|
||||
1.25 MB | 653.17 kB | 646.76 kB | 163.58 kB | 163.73 kB | three.js
|
||||
|
||||
2.14 MB | 726.71 kB | 724.14 kB | 180.25 kB | 181.07 kB | victory.js
|
||||
|
||||
3.20 MB | 1.01 MB | 1.01 MB | 332.13 kB | 331.56 kB | echarts.js
|
||||
|
||||
6.69 MB | 2.32 MB | 2.31 MB | 493.02 kB | 488.28 kB | antd.js
|
||||
6.69 MB | 2.32 MB | 2.31 MB | 493.01 kB | 488.28 kB | antd.js
|
||||
|
||||
10.95 MB | 3.51 MB | 3.49 MB | 910.11 kB | 915.50 kB | typescript.js
|
||||
10.95 MB | 3.51 MB | 3.49 MB | 910.12 kB | 915.50 kB | typescript.js
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue