================
@@ -911,7 +911,8 @@ void FactsGenerator::handleMovedArgsInCall(const
FunctionDecl *FD,
ArrayRef<const Expr *> Args) {
unsigned IsInstance = 0;
if (const auto *MD = dyn_cast<CXXMethodDecl>(FD);
- MD && MD->isInstance() && !isa<CXXConstructorDecl>(FD)) {
+ MD && !isa<CXXConstructorDecl>(FD) &&
+ MD->isImplicitObjectMemberFunction()) {
----------------
ojhunt wrote:
Right, but I'm looking at the tests in this PR
e.g. a hypothetical
```cpp
int get(this shared_ptr<NoMove> &&self, int)...
```
Or similar where there is a non-trivial/default move constructor? This is just
a paranoia question from me as this is a lifetime PR
https://github.com/llvm/llvm-project/pull/212154
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
