This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 5/9] Change decode_compound_collector to use std::vector
>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:
Simon> I've always read that it was bad to use std::move on a return value:
Simon> https://diego.assencio.com/?index=f57f25fd5a187c70fc7f34fcf5374773
Simon> though in this case it seems like the right thing to do.
It would be nice if gcc warned when a std::move was either redundant or
recommended. Maybe this is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86981
... I thought I heard that clang 8 has a new warning in this area but I
couldn't find it anywhere online, only a hint here:
https://bugzilla.mozilla.org/show_bug.cgi?id=1483985
So I'm not really sure how to file this.
Tom