fix corner case in side_effects (#5002)

fixes #5001
This commit is contained in:
Alex Lam S.L
2021-06-13 13:29:14 +01:00
committed by GitHub
parent f8b2215145
commit 6fc7a2ab6a
2 changed files with 38 additions and 5 deletions

View File

@@ -7596,11 +7596,22 @@ merge(Compressor.prototype, {
node.body = body;
}
} else if (body instanceof AST_Sequence) {
var tail = body.tail_node();
if (tail instanceof AST_Await && is_primitive(compressor, tail.expression)) {
tail = tail.expression.drop_side_effect_free(compressor);
body.expressions.pop();
if (tail) body.expressions.push(tail);
var exprs = body.expressions;
for (var i = exprs.length; --i >= 0;) {
var tail = exprs[i];
if (!(tail instanceof AST_Await)) break;
if (!is_primitive(compressor, tail.expression)) break;
if (exprs[i] = tail.expression.drop_side_effect_free(compressor)) break;
}
switch (i) {
case -1:
return make_node(AST_EmptyStatement, node);
case 0:
node.body = exprs[0];
break;
default:
exprs.length = i + 1;
break;
}
}
return node;

View File

@@ -1758,3 +1758,25 @@ issue_4987: {
]
node_version: ">=8"
}
issue_5001: {
options = {
awaits: true,
inline: true,
side_effects: true,
}
input: {
var a = 0;
(async function() {
a++ | await 42;
})();
console.log(a ? "PASS" : "FAIL");
}
expect: {
var a = 0;
void a++;
console.log(a ? "PASS" : "FAIL");
}
expect_stdout: "PASS"
node_version: ">=8"
}