DWARFDebugLoc(v4): Add an incremental parsing function
authorPavel Labath <pavel@labath.sk>
Tue, 5 Nov 2019 13:46:54 +0000 (14:46 +0100)
committerPavel Labath <pavel@labath.sk>
Fri, 15 Nov 2019 12:38:00 +0000 (13:38 +0100)
commit0908093977b2b80d00baa12f0b2f1424dde096fb
treefb4b665cad7f163289f1caa963afeb501a880406
parentc953e061b410163bc54771f186176a92aac04008
DWARFDebugLoc(v4): Add an incremental parsing function

Summary:
This adds a visitLocationList function to the DWARF v4 location lists,
similar to what already exists for DWARF v5. It follows the approach
outlined in previous patches (D69672), where the parsed form is always
stored in the DWARF v5 format, which makes it easier for generic code to
be built on top of that. v4 location lists are "upgraded" during
parsing, and then this upgrade is undone while dumping.

Both "inline" and section-based dumping is rewritten to reuse the
existing "generic" location list dumper. This means that the output
format is consistent for all location lists (the only thing one needs to
implement is the function which prints the "raw" form of a location
list), and that debug_loc dumping correctly processes base address
selection entries, etc.

The previous existing debug_loc functionality (e.g.,
parseOneLocationList) is rewritten on top of the new API, but it is not
removed as there is still code which uses them. This will be done in
follow-up patches, after I build the API to access the "interpreted"
location lists in a generic way (as that is what those users really
want).

Reviewers: dblaikie, probinson, JDevlieghere, aprantl, SouraVX

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69847
27 files changed:
llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h
llvm/include/llvm/DebugInfo/DWARF/DWARFUnit.h
llvm/lib/DebugInfo/DWARF/DWARFContext.cpp
llvm/lib/DebugInfo/DWARF/DWARFDebugLoc.cpp
llvm/lib/DebugInfo/DWARF/DWARFDie.cpp
llvm/lib/DebugInfo/DWARF/DWARFUnit.cpp
llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp
llvm/test/CodeGen/ARM/debug-info-sreg2.ll
llvm/test/DebugInfo/Sparc/subreg.ll
llvm/test/DebugInfo/X86/PR26148.ll
llvm/test/DebugInfo/X86/debug-loc-offset.mir
llvm/test/DebugInfo/X86/dw_op_minus_direct.ll
llvm/test/DebugInfo/X86/dwarfdump-debug-loc-error-cases.s
llvm/test/DebugInfo/X86/dwarfdump-debug-loc-error-cases2.s
llvm/test/DebugInfo/X86/dwarfdump-debug-loc-simple.test
llvm/test/DebugInfo/X86/live-debug-variables.ll
llvm/test/DebugInfo/X86/pieces-1.ll
llvm/test/DebugInfo/X86/sret.ll
llvm/test/DebugInfo/X86/stack-value-piece.ll
llvm/test/tools/dsymutil/X86/basic-lto-dw4-linking-x86.test
llvm/test/tools/dsymutil/X86/basic-lto-linking-x86.test
llvm/test/tools/dsymutil/X86/debug-loc-base-addr.test
llvm/test/tools/llvm-dwarfdump/X86/debug_loc-OP_addr.s
llvm/test/tools/llvm-dwarfdump/X86/debug_loc_base_address.s
llvm/test/tools/llvm-dwarfdump/X86/debug_loc_offset.test
llvm/test/tools/llvm-dwarfdump/X86/debugloc.s
llvm/tools/llvm-dwarfdump/Statistics.cpp