[PATCH] D98947: [Orc] Fix copy elision warning in RPCUtils

Stefan Gränitz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 19 07:16:18 PDT 2021


sgraenitz added a comment.

I actually didn't manage to reproduce the warning with GCC 10 on Debian, so I couldn't really test this. @thakis Do you want to run a pre-test? Otherwise, we could land it right away and see if it is effective.

@lhames Correct me if I am wrong, but I think `consumeAbandoned()` should have been called on `detail::ResultTraits<AltRetT>` and not on `detail::ResultTraits<typename Func::ReturnType>` in the first place. I simplified it now with `RTraits` analog to `appendCallNB()`. Hope it's fine.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98947/new/

https://reviews.llvm.org/D98947



More information about the llvm-commits mailing list