fix: avoid to visit nullptr in binaryen (#2943)

HerrCai0907 · web-flow · commit 7e2a62d9615d · 2025-08-28T21:45:56.000-07:00
diff --git a/src/compiler.ts b/src/compiler.ts
@@ -6764,7 +6764,7 @@ export class Compiler extends DiagnosticEmitter {
       if (thisType.isManaged) {
         let operand = operands[0];
         let precomp = module.runExpression(operand, ExpressionRunnerFlags.Default);
-        if (!isConstZero(precomp)) { // otherwise unnecessary
+        if (!precomp || !isConstZero(precomp)) { // otherwise unnecessary
           operands[operandIndex] = module.tostack(operand);
         }
       }
@@ -6778,7 +6778,7 @@ export class Compiler extends DiagnosticEmitter {
       if (paramType.isManaged) {
         let operand = operands[operandIndex];
         let precomp = module.runExpression(operand, ExpressionRunnerFlags.Default);
-        if (!isConstZero(precomp)) { // otherwise unnecessary
+        if (!precomp || !isConstZero(precomp)) { // otherwise unnecessary
           operands[operandIndex] = module.tostack(operand);
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -6764,7 +6764,7 @@ export class Compiler extends DiagnosticEmitter {`
`6764`	`6764`	`if (thisType.isManaged) {`
`6765`	`6765`	`let operand = operands[0];`
`6766`	`6766`	`let precomp = module.runExpression(operand, ExpressionRunnerFlags.Default);`
`6767`		`- if (!isConstZero(precomp)) { // otherwise unnecessary`
	`6767`	`+ if (!precomp \|\| !isConstZero(precomp)) { // otherwise unnecessary`
`6768`	`6768`	`operands[operandIndex] = module.tostack(operand);`
`6769`	`6769`	`}`
`6770`	`6770`	`}`
`@@ -6778,7 +6778,7 @@ export class Compiler extends DiagnosticEmitter {`
`6778`	`6778`	`if (paramType.isManaged) {`
`6779`	`6779`	`let operand = operands[operandIndex];`
`6780`	`6780`	`let precomp = module.runExpression(operand, ExpressionRunnerFlags.Default);`
`6781`		`- if (!isConstZero(precomp)) { // otherwise unnecessary`
	`6781`	`+ if (!precomp \|\| !isConstZero(precomp)) { // otherwise unnecessary`
`6782`	`6782`	`operands[operandIndex] = module.tostack(operand);`
`6783`	`6783`	`}`
`6784`	`6784`	`}`