[clang] [LifetimeSafety] Treat std::unique_ptr::release() as a move operation (PR #180230)

[Gábor Horváth via cfe-commits]

================
@@ -255,6 +255,26 @@ template <typename T> static bool isRecordWithAttr(QualType Type) {
 bool isGslPointerType(QualType QT) { return isRecordWithAttr<PointerAttr>(QT); }
 bool isGslOwnerType(QualType QT) { return isRecordWithAttr<OwnerAttr>(QT); }
 
+static bool isStdUniquePtr(const CXXRecordDecl *RD) {
----------------
Xazax-hun wrote:

The call site seems to be passing in the result of `getParent()` which can never be null. Should this take a reference instead of having a null check?

https://github.com/llvm/llvm-project/pull/180230