Comment explaining need for std::move

src/storm/modelchecker/prctl/helper/SparseMdpPrctlHelper.cpp

@@ -682,7 +682,7 @@ namespace storm {
                     for (auto& element : result) {
                         element = storm::utility::one<ValueType>() - element;
                     }
-                    return std::move(result);
+                    return std::move(result); // move() required by, e.g., clang 3.8
                 }
             }
             

src/storm/modelchecker/reachability/SparseDtmcEliminationModelChecker.cpp

@@ -494,7 +494,7 @@ namespace storm {
                 for (auto state : ~maybeStates | initialStates) {
                     (*checkResult)[state] = result[state];
                 }
-                return std::move(checkResult);
+                return std::move(checkResult); // move() required by, e.g., clang 3.8
             }
             return std::make_unique<ExplicitQuantitativeCheckResult<ValueType>>(result);
         }
@@ -591,7 +591,7 @@ namespace storm {
                 for (auto state : ~maybeStates | initialStates) {
                     (*checkResult)[state] = result[state];
                 }
-                return std::move(checkResult);
+                return std::move(checkResult); // move() required by, e.g., clang 3.8
             }
             return std::make_unique<ExplicitQuantitativeCheckResult<ValueType>>(result);
         }