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

[Wolfgang Pieb via Phabricator via llvm-commits]

wolfgangp added a comment.



> If it's hard-coded to anything other than 4 or 8 the constructor for DWARFExpression would assert. If it's hard-coded to 4 it would work just fine (except for the DW_OP_call_ref example with an 8-byte operand mentioned above, and that would probably just report an obscure error and move on to the next list).

In fact, the 8-byte operand would only be valid for DWARF64, which we don't support at the moment, so the bottom line is I can't see anything that would be incorrectly handled with a hard-coded 4.


https://reviews.llvm.org/D52444