clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
We can't currently do this because we have a dependency on having this be there
for existing customers. The problem is that the debug info always has the base
names of types in the tables: "pointer_type", "reference_type", etc. The
problem arises when we say 'I want only matches to the full qualified name of
"size_type"'. We will pick up many many matches if we don't look up
"::size_type" because every STL class has things like:
template <class _Tp, class _Allocator = allocator<_Tp> >
class vector : private __vector_base<_Tp, _Allocator>
{
private:
typedef __vector_base<_Tp, _Allocator> __base;
typedef allocator<_Tp> __default_allocator_type;
public:
typedef vector __self;
typedef _Tp value_type;
typedef _Allocator allocator_type;
typedef typename __base::__alloc_traits __alloc_traits;
typedef typename __base::reference reference;
typedef typename __base::const_reference const_reference;
typedef typename __base::size_type size_type;
typedef typename __base::difference_type difference_type;
So having the extra "::" on the name for things that are top level helps us
avoid pulling in many mega bytes of DWARF every time do this. So we can't make
these changes without affecting type lookups. The DWARFDDeclContext class is
used to get fully qualified names during SymbolFileDWARF::FindTypes(...) to
avoid pulling in all DWARF for any type whose basename matches, but this only
happens when using the Apple accelerator tables. For a type like like:
typedef unsigned long size_type;
The Apple accelerator tables will have an entry for "size_type", and this table
entry includes a fully qualified name hash. So we will hash up "::size_type"
and include that in the type accelerator table entry so we can omit any non
matching entries (all "size_type" values in all STL classes will have
"std::vector::size_type" hashed up in their "size_type" accelerator table entry
and we can avoid parsing all of the DWARF for std::vector and all other STL
classes when).
So I can't condone this patch without further modifications.
https://reviews.llvm.org/D22369
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
