Code cleanup: Remove std::move() around xvalue (NFC)
authorLogan Chien <tzuhsiang.chien@gmail.com>
Mon, 22 Jun 2015 23:16:02 +0000 (23:16 +0000)
committerLogan Chien <tzuhsiang.chien@gmail.com>
Mon, 22 Jun 2015 23:16:02 +0000 (23:16 +0000)
Remove std::move() around xvalue so that copy elision is eligible.
In case that copy elision is not appliable, the c++ standard also
guarantees the move semantics on xvalue.  Thus, it is not necessary
to wrap Args with std::move.

This also silence a warning since r240345.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@240355 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Option/OptTable.cpp

index 2160b15c1cf05d76c131080d3ac4b3e8ee4da444..185a734f8f5f24a946489a6c5e564e5e195e6ef4 100644 (file)
@@ -287,7 +287,7 @@ InputArgList OptTable::ParseArgs(ArrayRef<const char *> ArgArr,
     Args.append(A);
   }
 
-  return std::move(Args);
+  return Args;
 }
 
 static std::string getOptionHelpName(const OptTable &Opts, OptSpecifier Id) {