[PATCH] D66011: [llvm-readobj] - Remove 'error(Error EC)' helper.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 13 04:01:14 PDT 2019


jhenderson added inline comments.


================
Comment at: tools/llvm-readobj/COFFDumper.cpp:1170
 
   if (auto EC = CVSD.dump(Symbols)) {
     W.flush();
----------------
Is EC an Error or std::error_code? If the latter, the move below is unnecessary.


================
Comment at: tools/llvm-readobj/COFFDumper.cpp:1271
                                       Types, Hashes, PCHSignature))
-          return error(std::move(EC));
+          return reportError(std::move(EC), Obj->getFileName());
       } else {
----------------
Same comment here and immediately below as above.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66011/new/

https://reviews.llvm.org/D66011





More information about the llvm-commits mailing list