[clang] [WebKit Checkers] Trivial analysis should check destructors of function parameters and local variables (PR #181576)

[Balázs Benics via cfe-commits]

================
@@ -516,19 +516,35 @@ class TrivialFunctionAnalysisVisitor
     return Result;
   }
 
+  bool CanTriviallyDestruct(const Type *T) {
+    if (T->isIntegralOrEnumerationType())
+      return true;
+    if (isa<PointerType>(T) || T->isNullPtrType())
----------------
steakhal wrote:

What is the purpose of `T->isNullPtrType()` Its so arbitrary. Why would anyone have a `nullptr_t` as a parameter type?
I guess it's true that it can trivially destruct, fair, but was it actually used?

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