[PATCH] D53155: [DWARF] Make llvm-dwarfdump display location lists in a .dwp file correctly. Fixes PR38990.

[Wolfgang Pieb via Phabricator via llvm-commits]

wolfgangp added inline comments.


================
Comment at: include/llvm/DebugInfo/DWARF/DWARFUnit.h:203
+  /// refer to substrings of it when opportune (e.g. in a DWARF package file).
+  std::pair<const DWARFSection *, StringRef> LocSection;
   const DWARFSection &LineSection;
----------------
dblaikie wrote:
> Could this be a union, perhaps?
Yes, that's better. I guess we can't use std::variant yet?


https://reviews.llvm.org/D53155