[XCOFF] Fix the program abortion issue.

[Esme]

Summary

Use std::move(E) to avoid Program aborted due to an unhandled Error:..
This is found during testing the error paths in D98003.
See the error case 1 in llvm/test/tools/obj2yaml/XCOFF/invalid.yaml of D98003.

[jhenderson]

LGTM.

llvm/lib/Object/XCOFFObjectFile.cpp

296

Fix this in a separate NFC commit. No need for a review.

llvm/lib/Object/XCOFFObjectFile.cpp

if (isSectionVirtual(Sec))
     return ArrayRef<uint8_t>();
 
   uint64_t OffsetToRaw;
   if (is64Bit())
     OffsetToRaw = toSection64(Sec)->FileOffsetToRawData;
   else
     OffsetToRaw = toSection32(Sec)->FileOffsetToRawData;
 
   const uint8_t * ContentStart = base() + OffsetToRaw;
   uint64_t SectionSize = getSectionSize(Sec);
-  if (checkOffset(Data, reinterpret_cast<uintptr_t>(ContentStart), SectionSize))
-    return make_error<BinaryError>();
+  if (Error E = Binary::checkOffset(
+          Data, reinterpret_cast<uintptr_t>(ContentStart), SectionSize))
+    return std::move(E);
 
   return makeArrayRef(ContentStart,SectionSize);
 }
 
 uint64_t XCOFFObjectFile::getSectionAlignment(DataRefImpl Sec) const {
   uint64_t Result = 0;
   llvm_unreachable("Not yet implemented!");
   return Result;