llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> `!E->getType()->isAnyComplexType()` is implied by `!canClassify()`. --- Full diff: https://github.com/llvm/llvm-project/pull/156666.diff 1 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+1-2) ``````````diff diff --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp index 56552f3969216..89ff5d27a4143 100644 --- a/clang/lib/AST/ByteCode/Compiler.cpp +++ b/clang/lib/AST/ByteCode/Compiler.cpp @@ -4199,8 +4199,7 @@ template <class Emitter> bool Compiler<Emitter>::visit(const Expr *E) { return this->discard(E); // Create local variable to hold the return value. - if (!E->isGLValue() && !E->getType()->isAnyComplexType() && - !canClassify(E->getType())) { + if (!E->isGLValue() && !canClassify(E->getType())) { UnsignedOrNone LocalIndex = allocateLocal(E); if (!LocalIndex) return false; `````````` </details> https://github.com/llvm/llvm-project/pull/156666 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
