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

[Ryosuke Niwa via cfe-commits]

================
@@ -92,7 +92,18 @@ class NoDeleteChecker : public Checker<check::ASTDecl<TranslationUnitDecl>> {
       return;
 
     auto Body = FD->getBody();
-    if (!Body || TFA.isTrivial(Body))
+    if (!Body)
+      return;
+
+    bool ParamHaveTrivialDtors = true;
+    for (auto *Param : FD->parameters()) {
+      if (!TFA.hasTrivialDtor(Param)) {
+        ParamHaveTrivialDtors = false;
+        break;
+      }
+    }
+
+    if (ParamHaveTrivialDtors && TFA.isTrivial(Body))
----------------
rniwa wrote:

Nice!

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