[clang][Interp] Return false from visitExpr() if allocateLocal failed

tbaederr · tbaederr · commit 67c18721eb21 · 2024-05-13T16:59:30.000+02:00
diff --git a/clang/lib/AST/Interp/ByteCodeExprGen.cpp b/clang/lib/AST/Interp/ByteCodeExprGen.cpp
@@ -2861,7 +2861,8 @@ bool ByteCodeExprGen<Emitter>::visitExpr(const Expr *E) {
     return this->emitRetValue(E) && RootScope.destroyLocals();
   }
 
-  return RootScope.destroyLocals();
+  RootScope.destroyLocals();
+  return false;
 }
 
 /// Toplevel visitDecl().

Original file line number	Diff line number	Diff line change
`@@ -2861,7 +2861,8 @@ bool ByteCodeExprGen<Emitter>::visitExpr(const Expr *E) {`
`2861`	`2861`	`return this->emitRetValue(E) && RootScope.destroyLocals();`
`2862`	`2862`	`}`
`2863`	`2863`
`2864`		`- return RootScope.destroyLocals();`
	`2864`	`+ RootScope.destroyLocals();`
	`2865`	`+ return false;`
`2865`	`2866`	`}`
`2866`	`2867`
`2867`	`2868`	`/// Toplevel visitDecl().`