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

[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());
----------------
Serosh-commits wrote:

I have simplified it to just the RE->containsErrors() check as you suggested and kept it within the original block structure check it 

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