timon-ul wrote: > When a class C has a unique_ptr<T> field and you forward declare T, then you > need to move C::~C()=default to the cpp, heaving it in the header would cause > linker issues. So that's a use case where you want to keep the default > keyword I think.
I see, thank you so much, I wasn't aware of this. Then I would drop that requirement for this PR. What I just remembered though which should be added in this PR is a comment in the release notes `clang-tools-extra/docs/ReleaseNotes.rst` describing the new feature. https://github.com/llvm/llvm-project/pull/71950 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
