llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/157999.diff 1 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+9-11) ``````````diff diff --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp index a213583382509..9d517aa72267e 100644 --- a/clang/lib/AST/ByteCode/Compiler.cpp +++ b/clang/lib/AST/ByteCode/Compiler.cpp @@ -2949,19 +2949,17 @@ bool Compiler<Emitter>::VisitMaterializeTemporaryExpr( if (!this->emitSetLocal(*SubExprT, LocalIndex, E)) return false; return this->emitGetPtrLocal(LocalIndex, E); - } else { + } - if (!this->checkLiteralType(SubExpr)) + if (!this->checkLiteralType(SubExpr)) + return false; + const Expr *Inner = E->getSubExpr()->skipRValueSubobjectAdjustments(); + if (UnsignedOrNone LocalIndex = + allocateLocal(E, Inner->getType(), E->getExtendingDecl())) { + InitLinkScope<Emitter> ILS(this, InitLink::Temp(*LocalIndex)); + if (!this->emitGetPtrLocal(*LocalIndex, E)) return false; - - const Expr *Inner = E->getSubExpr()->skipRValueSubobjectAdjustments(); - if (UnsignedOrNone LocalIndex = - allocateLocal(E, Inner->getType(), E->getExtendingDecl())) { - InitLinkScope<Emitter> ILS(this, InitLink::Temp(*LocalIndex)); - if (!this->emitGetPtrLocal(*LocalIndex, E)) - return false; - return this->visitInitializer(SubExpr) && this->emitFinishInit(E); - } + return this->visitInitializer(SubExpr) && this->emitFinishInit(E); } return false; } `````````` </details> https://github.com/llvm/llvm-project/pull/157999 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
