[PATCH] D73086: [WIP][DWARF5] Added support for debug_macro section parsing and dumping in llvm-dwarfdump.

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

This might be a legacy thing, but it looks like `DWARFSection` is just a wrapper around a `StringRef`. Could they all use `StringRef` (or `DWARFSection`), instead of having some use one and some use the other? That might require an additional change or two, but it seems pointless having the two types to me.


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

https://reviews.llvm.org/D73086