[PATCH] D52444: [DWARF] Make llvm-dwarfdump display the .debug_loc.dwo section (fixes PR38991)

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In https://reviews.llvm.org/D52444#1245772, @wolfgangp wrote:

> One reason why we may still want addrsize is so we can skip the correct amount of bytes during parsing if we do find a DW_OP_addr in a DWO (and report an error).


I think it'd be reasonable to report an error and stop parsing - skipping the rest of the location list? The value is out of place in the DWO section, so I'm not sure it's reasonable to conclude what length it should be, etc.


https://reviews.llvm.org/D52444