Review Request: Patch to reduce C++ memory consumption by removing DuContext* pointer from the AST

Alexander Dymo alexander.dymo at gmail.com
Mon Feb 15 22:25:10 UTC 2010


2010/2/15 David Nolden <zwabel at googlemail.com>:
> Although it might be faster if you use a google::sparse_hash (see its usage in
> duchain.cpp) instead of QMap.

I tried the dense_hash_map (which should IIRC be the fastest one) with
a simple hash object which just casts AST* pointer to the size_t. It
didn't improve the performance:

declaration and use building for kdevelop project:
pointer:		41.77
QMap:		42.55
dense_hash:	42.52

So, I'd say that I'll not apply this patch.




More information about the KDevelop-devel mailing list