From 3509503e18e34127a3868c18c15f5be64b389233 Mon Sep 17 00:00:00 2001
From: peter klausler <pklausler@nvidia.com>
Date: Mon, 9 Apr 2018 12:51:33 -0700
Subject: [PATCH] [flang] Remove some needless std::move() calls for
 readability.

Original-commit: flang-compiler/f18@88aadf45dccb6c7fe0443345842d6ecced2258bd
Reviewed-on: https://github.com/flang-compiler/f18/pull/48
Tree-same-pre-rewrite: false
---
 flang/lib/parser/basic-parsers.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/flang/lib/parser/basic-parsers.h b/flang/lib/parser/basic-parsers.h
index f63c424..d543e55 100644
--- a/flang/lib/parser/basic-parsers.h
+++ b/flang/lib/parser/basic-parsers.h
@@ -212,7 +212,7 @@ public:
   std::optional<resultType> Parse(ParseState *state) const {
     if (std::optional<resultType> ax{pa_.Parse(state)}) {
       if (pb_.Parse(state)) {
-        return std::move(ax);
+        return ax;
       }
     }
     return {};
@@ -1188,7 +1188,7 @@ constexpr struct NextCh {
   constexpr NextCh() {}
   std::optional<const char *> Parse(ParseState *state) const {
     if (std::optional<const char *> result{state->GetNextChar()}) {
-      return std::move(result);
+      return result;
     }
     state->Say("end of file"_err_en_US);
     return {};
-- 
2.7.4