[PATCH] D49964: [DWARF] Refactor DWARF classes to use unified error reporting. NFC.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 2 02:46:08 PDT 2018


JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

In the differential you linked we agreed to have the API include an error code as an argument. That still leaves us the option to pass an `inconvertibleErrorCode()` as the first argument.

Anyway, since nobody else seems to mind the switch to actual error codes, this LGTM. Thanks for taking the time to work on this, Victor!


Repository:
  rL LLVM

https://reviews.llvm.org/D49964





More information about the llvm-commits mailing list