[clang] [Clang] FunctionEffect analysis was missing a CXXBindTemporaryExpr's implicit call to a destructor. (PR #166110)

[Doug Wyatt via cfe-commits]

dougsonos wrote:

> Obviously no-one writes code like this, but even if you remove some of the unnecessary parts here, the AST is still sufficiently complex to where checking all possible patterns becomes annoying.

OK, I'm inclined to think the rare duplicate warnings are tolerable for now (they're at least at different source locations), even if they're not correct.

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