This revision was automatically updated to reflect the committed changes.
Closed by commit rL327809: DWARFUnit split out of DWARFCompileUnit (authored by
jankratochvil, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
jankratochvil added a comment.
In https://reviews.llvm.org/D40466#1034875, @davide wrote:
> This is No functional change, right (just code churn)?
Right.
https://reviews.llvm.org/D40466
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
davide accepted this revision.
davide added a comment.
This is No functional change, right (just code churn)? If so, LGTM.
https://reviews.llvm.org/D40466
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.
This is fine. Lets start with this and then worry about each next patch. 02/11
next.
https://reviews.llvm.org/D40466
___
lldb-commits
jankratochvil updated this revision to Diff 137961.
jankratochvil added a comment.
bugfix
https://reviews.llvm.org/D40466
Files:
source/Plugins/SymbolFile/DWARF/CMakeLists.txt
source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.cpp
source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.h
