<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 4, 2015 at 10:38 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Sat, May 2, 2015 at 11:47 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><span><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 29, 2015 at 2:41 AM, Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@gmail.com" target="_blank">benny.kra@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span>On Wed, Apr 29, 2015 at 5:59 AM, Richard Trieu <<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>> wrote:<br>
> Since this warning did not exist when std::move was first introduced, and<br>
> many people are not familiar with copy elision rules, there have been a lot<br>
> of uses of the pessimizing move pattern.  In relative numbers, the<br>
> pessimizing move warning is over 5x more common than either the range loop<br>
> warning (-Wrange-loop-analysis) or the proposed comma warning (-Wcomma).<br>
> With such a high occurrence, it seemed better not to have it on by default.<br>
<br>
</span>And every single occurence is a potential performance issue and easy<br>
to fix. While this may be too noisy for default, it's definitely a<br>
candidate for -Wall imho.<br></blockquote></div><br></div></span><div class="gmail_extra">+1. As-is, nobody will find this warning. And in general, the bar for warnings is "useful enough to be in -Wall, or it shouldn't be there", right? (Which this warning definitely is.)</div></div></blockquote></div></div><div><br>I think the bar at least used to be "useful enough to be on by default or it shouldn't be there" (the big exception in any analysis based warning because building the CFG costs some non-trivial % - so we keep /all/ of those off by default). Doug particularly espoused this notion & I've mostly been promoting it since then on the assumption that that was still the goal - though I don't agree with it entirely myself.<span class=""><font color="#888888"><br><br>- David<br> </font></span></div></div></div></div></blockquote><div>I created a patch (<a href="http://reviews.llvm.org/D9493">http://reviews.llvm.org/D9493</a>) to place the std::move warnings into -Wmove and put that warning group into -Wmost, which gets pulled into -Wall. </div></div></div></div>