[PATCH] D61421: [Object] Change getSectionName() to use Expected<StringRef>

[Fangrui Song via Phabricator via llvm-commits]

MaskRay marked 2 inline comments as done.
MaskRay added inline comments.


================
Comment at: llvm/tools/llvm-readobj/COFFDumper.cpp:1545
                                        COFF::IMAGE_FILE_MACHINE_ARM64);
-    Decoder.dumpProcedureData(*Obj);
+    (void)!Decoder.dumpProcedureData(*Obj);
     break;
----------------
jhenderson wrote:
> I'm not sure this is the right way to throw away an Error. Assuming for a moment that we don't want to report this error here (I'm guessing we probably should at some point), you should use something like `consumeError`.
Thanks! I forgot `consumeError`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61421