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