================ @@ -453,11 +481,11 @@ BuiltinTypeMethodBuilder &BuiltinTypeMethodBuilder::dereference(T Ptr) { return *this; } -BuiltinTypeDeclBuilder &BuiltinTypeMethodBuilder::finalizeMethod() { +BuiltinTypeDeclBuilder &BuiltinTypeMethodBuilder::finalize() { assert(!DeclBuilder.Record->isCompleteDefinition() && "record is already complete"); - assert(Method != nullptr && - "method decl not created; are you missing a call to build the body?"); + + ensureCompleteDecl(); ---------------- hekota wrote:
I don't think it is necessary since it is exactly what `ensureCompleteDecl` is doing - making sure Method is created. https://github.com/llvm/llvm-project/pull/131384 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits