llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> `Func` is only null for the bottom frame, which we create using another constructor. --- Full diff: https://github.com/llvm/llvm-project/pull/220284.diff 1 Files Affected: - (modified) clang/lib/AST/ByteCode/InterpFrame.cpp (+1-3) ``````````diff diff --git a/clang/lib/AST/ByteCode/InterpFrame.cpp b/clang/lib/AST/ByteCode/InterpFrame.cpp index 033e35eee42ad..b895dee3ae1ec 100644 --- a/clang/lib/AST/ByteCode/InterpFrame.cpp +++ b/clang/lib/AST/ByteCode/InterpFrame.cpp @@ -32,13 +32,11 @@ InterpFrame::InterpFrame(InterpState &S, const Function *Func, : Caller(Caller), S(S), Func(Func), RetPC(RetPC), Args(static_cast<char *>(S.Stk.top())), ArgSize(ArgSize), Depth(Caller ? Caller->Depth + 1 : 0) { + assert(Func); #ifndef NDEBUG FrameOffset = S.Stk.size(); #endif - if (!Func) - return; - FuncFlags |= Func->hasRVO() * HasRVOFlag; FuncFlags |= Func->hasThisPointer() * HasThisFlag; `````````` </details> https://github.com/llvm/llvm-project/pull/220284 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
