[PATCH] D43322: Diagnose cases of "return x" that should be "return std::move(x)" for efficiency
Arthur O'Dwyer via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 14 14:57:51 PDT 2018
Quuxplusone updated this revision to Diff 138457.
Quuxplusone added a comment.
Boldly add `-Wreturn-std-move` to `-Wmove` (and thus also to `-Wall`).
The backward-compatibility-concerned diagnostic, `-Wreturn-std-move-in-c++11`, is *not* appropriate for `-Wmove`; but I believe this main diagnostic is appropriate.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 34291 bytes
Desc: not available
More information about the cfe-commits