vbvictor wrote:

I'm -1 on separating this checks because we essentially will duplicate a huge 
portion of logic into multiple places which can double the time of analysis and 
we need to keep two versions in sync.

One thing we can do is provide better diag message like:
```
warning: 'X' used after it was invalidated by 'MyFunction' call 
[use-after-move].
note: invalidation occurred here
```
So we clearly show that it's not `std::move/std::forward` but a custom function.

https://github.com/llvm/llvm-project/pull/170346
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to