[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