LogicalExpression tree shaking optimisation breaks with comment

[tanhauhau]

• Rollup Version: 2.26.9
• Operating System (or Browser): chrome
• Node Version (if applicable):
• Link to reproduction (IMPORTANT, read below):

function foo() {
	return true && // comment
		1;
}

function bar() {
	return false || // comment
		1;
}


function baz() {
	return null ?? // comment
		1;
}
	
console.log(foo());
console.log(bar());
console.log(baz());

Rollup REPL

Expected Behavior

should print out '1' for all cases

Actual Behavior

prints out undefined

Rollup outputs:

function foo() {
	return  // comment		1;
}

function bar() {
	return  // comment		1;
}


function baz() {
	return  // comment		1;
}
	
console.log(foo());
console.log(bar());
console.log(baz());

[tanhauhau]

related: #3734, #3035

[lukastaegert]

Thanks for spotting. So it is actually the original ASI PR #3035 that broke this, I will have a look how to fix this. So I believe the solution must be to either remove line comments completely or turn them into block comments in such situations as they always imply a line-break. I'm leaning towards removing them for now as the simple solution.

[lukastaegert]

Fix at #3762