[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