friss created this revision.
friss added reviewers: clayborg, jingham.
Herald added a subscriber: JDevlieghere.
We would fail to resolve (and thus display the value of) any
templated type which contained a template template argument even
though we don't really use template arguments.
This patch a
This revision was automatically updated to reflect the committed changes.
Closed by commit rL327810: Move the codebase to use: DWARFCompileUnit ->
DWARFUnit (authored by jankratochvil, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D42
Author: jankratochvil
Date: Sun Mar 18 13:11:02 2018
New Revision: 327810
URL: http://llvm.org/viewvc/llvm-project?rev=327810&view=rev
Log:
Move the codebase to use: DWARFCompileUnit -> DWARFUnit
Now the codebase can use the DWARFUnit superclass. It will make it later
seamlessly work also with DW
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:
https://reviews.llvm.org/D40466?vs=137961&id
Author: jankratochvil
Date: Sun Mar 18 13:09:02 2018
New Revision: 327809
URL: http://llvm.org/viewvc/llvm-project?rev=327809&view=rev
Log:
DWARFUnit split out of DWARFCompileUnit
DW_TAG_partial_unit for DWZ can be then presented by DWARFPartialUnit also
inherited from DWARFUnit.
Differential re
jankratochvil added inline comments.
Comment at: source/Plugins/SymbolFile/DWARF/DWARFUnit.h:34
class DWARFUnit {
+ friend class DWARFCompileUnit;
+
clayborg wrote:
> DWARFCompileUnit inherits from this class. This isn't needed right?
It is needed now as `DWARF
jankratochvil updated this revision to Diff 138860.
https://reviews.llvm.org/D40474
Files:
include/lldb/Utility/ConstString.h
include/lldb/Utility/FileSpec.h
source/Plugins/SymbolFile/DWARF/CMakeLists.txt
source/Plugins/SymbolFile/DWARF/DWARFCompileUnit.cpp
source/Plugins/SymbolFile/DWA
