[clang] [clang][bytecode] Fix crash in void functions returning non-void expr… (PR #176550)

[Timm Baeder via cfe-commits]

================
@@ -5678,8 +5678,9 @@ bool Compiler<Emitter>::visitReturnStmt(const ReturnStmt *RS) {
       return this->emitRet(*ReturnType, RS);
     }
 
-    if (RE->getType()->isVoidType()) {
-      if (!this->visit(RE))
+    if (RE->getType()->isVoidType() ||
+        (CompilingFunction && CompilingFunction->getReturnType()->isVoidType())) {
+      if (!this->discard(RE))
         return false;
     } else {
       InitLinkScope<Emitter> ILS(this, InitLink::RVO());
----------------
tbaederr wrote:

I'd rather just add a `RE->containsErrors()` check here

https://github.com/llvm/llvm-project/pull/176550