[PATCH] D34405: [Testing/Support] Remove the const_cast in TakeExpected

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 09:25:02 PDT 2017


zturner added inline comments.


================
Comment at: include/llvm/Testing/Support/Error.h:34
+template <typename T> ExpectedHolder<T> TakeExpected(Expected<T> &&Exp) {
+  return TakeExpected(Exp);
 }
----------------
Should this be `std::move(Exp)`?


https://reviews.llvm.org/D34405





More information about the llvm-commits mailing list