[PATCH] D74481: [Debuginfo][NFC] Create common error handlers for DWARFContext.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 12 10:33:47 PST 2020


JDevlieghere requested changes to this revision.
JDevlieghere added a comment.
This revision now requires changes to proceed.

I think we should pass the error handles as part of the DIDumpOptions to the dump methods. That way we have default handlers for when no options are specified and they're inherited when the options are set explicitly. It also has the benefit of not needlessly complicating the interfaces.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74481





More information about the llvm-commits mailing list