[PATCH] D48349: [ADT] Add llvm::unique_function which is like std::function but supporting move-only closures.
Dean Michael Berris via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 20 20:44:10 PDT 2018
dberris added inline comments.
================
Comment at: llvm/include/llvm/ADT/function.h:187
+ ~unique_function();
+ new (this) unique_function(std::move(RHS));
+ return *this;
----------------
Do you still need the `std::move(...)` here?
Repository:
rL LLVM
https://reviews.llvm.org/D48349
More information about the llvm-commits
mailing list