diff --git a/folly/futures/Future-inl.h b/folly/futures/Future-inl.h
index fb110d0..86883c6 100644
--- a/folly/futures/Future-inl.h
+++ b/folly/futures/Future-inl.h
@@ -602,8 +602,8 @@ template <class, class, typename = void> struct CollectContextHelper;
 template <class T, class VecT>
 struct CollectContextHelper<T, VecT,
     typename std::enable_if<std::is_same<T, VecT>::value>::type> {
-  static inline std::vector<T>& getResults(std::vector<VecT>& results) {
-    return results;
+  static inline std::vector<T>&& getResults(std::vector<VecT>& results) {
+    return std::move(results);
   }
 };
 
diff --git a/folly/futures/test/FutureTest.cpp b/folly/futures/test/FutureTest.cpp
index edc9d30..08f13af 100644
--- a/folly/futures/test/FutureTest.cpp
+++ b/folly/futures/test/FutureTest.cpp
@@ -877,6 +877,18 @@ TEST(Future, collect) {
 
     EXPECT_THROW(allf.value(), eggs_t);
   }
+
+  // move only compiles
+  {
+    vector<Promise<unique_ptr<int>>> promises(10);
+    vector<Future<unique_ptr<int>>> futures;
+
+    for (auto& p : promises)
+      futures.push_back(p.getFuture());
+
+    collect(futures.begin(), futures.end());
+  }
+
 }
 
 struct NotDefaultConstructible {