[PATCH] D85505: [dwarf2yaml] Change the return type of dumping functions to Error.

[Xing GUO via Phabricator via llvm-commits]

Higuoxing added a comment.

In D85505#2206076 <https://reviews.llvm.org/D85505#2206076>, @jhenderson wrote:

> I like that it makes this makes all the `dumpDebug*` functions consistent. However, are there going to actually be changes to all of these functions to make the new error handling path useful?

This change is intended to make `elf2yaml` and `macho2yaml` share the same logic when dumping DWARF sections. If the parser fails, we are able to dump them as raw content sections.

I think not all of the error handling path are useful. For example, the .debug_str section's parser never fails.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85505