[PATCH] D52008: [analyzer] Handle forwarding reference better in ExprMutationAnalyzer.

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added a comment.

In https://reviews.llvm.org/D52008#1233667, @shuaiwang wrote:

> Just some quick comments, I'll take a deeper look into other comments later.
>
> This diff along unfortunately won't be able to handle `emplace_back` just yet


My apologies, for some reason i though it was supposed to.
Once again, thank you for working on this!

> , the reason (I believe, haven't fully tested) is that `std::forward` is not handled properly and almost all std functions involving forwarding references at least `std::forward`'ed once.
>  I have some more changes locally that treats `std::move` and `std::forward` just as casts and that should be able to really push the analysis further down the forwarding chain instead of stopping at `std::forward` call.



> Rephased diff description to be more clear. Sorry for the confusion.

No problem!


Repository:
  rC Clang

https://reviews.llvm.org/D52008