================
@@ -5678,11 +5678,10 @@ bool Compiler<Emitter>::visitReturnStmt(const
ReturnStmt *RS) {
return this->emitRet(*ReturnType, RS);
}
- if (RE->getType()->isVoidType()) {
- if (!this->visit(RE))
+ if (RE->containsErrors() || RE->getType()->isVoidType()) {
+ if (!this->discard(RE))
return false;
} else {
- InitLinkScope<Emitter> ILS(this, InitLink::RVO());
// RVO - construct the value in the return location.
----------------
tbaederr wrote:
Why are ou
1) Not moving the `containsErrors()` check in the else block of the `if
(RE->getType()->isVoidType()` as suggested?
2) Replacing the `visit()` with a `discard()`?
3) Removing the `InitLinkScope`?
https://github.com/llvm/llvm-project/pull/176550
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
