[PATCH] D54097: Save memory in ASTReader by using DenseMap instead of vector

2018-11-17 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev added a comment. Ping... https://reviews.llvm.org/D54097 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D54097: Save memory in ASTReader by using DenseMap instead of vector

2018-11-05 Thread Yuka Takahashi via Phabricator via cfe-commits
yamaguchi added a comment. FYI, raw benchmarking data: https://gist.github.com/yamaguchi1024/4f28406f51b413129cc762365fdd76c8 https://gist.github.com/yamaguchi1024/318b30db03adb10b7b230b706012a14c https://reviews.llvm.org/D54097 ___ cfe-commits mail

[PATCH] D54097: Save memory in ASTReader by using DenseMap instead of vector

2018-11-05 Thread Yuka Takahashi via Phabricator via cfe-commits
yamaguchi created this revision. yamaguchi added a reviewer: rsmith. ReadASTBlock was allocating extra memory by resizing vector for all record decls, and half of them stayed nullptr without being loaded. For us this part was crucial and this patch had large amount of memory improvement. About cl