[PATCH] D65774: Removing redundant-move warnings generated by gcc9x.
Zahira Ammarguellat via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 8 05:04:45 PDT 2019
zahiraam added a comment.
In D65774#1619537 <https://reviews.llvm.org/D65774#1619537>, @rupprecht wrote:
> Agree with all these comments -- I'm no expert on when and when not to std::move, but this is undoing changes I've made to fix buildbots on different compilers.
I agree. This is probably more aggressive than I anticipated. I am looking at them and trying to figure out which and if some of them can stay.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65774/new/
https://reviews.llvm.org/D65774
More information about the llvm-commits
mailing list