[PATCH] D12839: Extend MoveConstructorInitCheck to also flag constructor arguments passed by value and can be moved assigned to fields.
Felix Berger via cfe-commits
cfe-commits at lists.llvm.org
Sun Sep 27 14:30:27 PDT 2015
flx updated this revision to Diff 35826.
flx marked 7 inline comments as done.
flx added a comment.
I changed the check to also produce a fix that wraps the argument in std::move().
When I modified the test include -isystem %S/Inputs/Headers it broke and only produces warnings but no fixes anymore. Is there a subtle difference in how tests with includes need to be invoked?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 16897 bytes
Desc: not available
More information about the cfe-commits