Re: [PATCH] D16395: [Coverage] Reduce complexity of adding function mapping records

2016-01-21 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL258424: [Coverage] Reduce complexity of adding function mapping records (authored by vedantk). Changed prior to commit: http://reviews.llvm.org/D16395?vs=45491&id=45574#toc Repository: rL LLVM http:

Re: [PATCH] D16395: [Coverage] Reduce complexity of adding function mapping records

2016-01-21 Thread David Li via cfe-commits
davidxl accepted this revision. davidxl added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D16395 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-

[PATCH] D16395: [Coverage] Reduce complexity of adding function mapping records

2016-01-20 Thread Vedant Kumar via cfe-commits
vsk created this revision. vsk added a reviewer: davidxl. vsk added a subscriber: cfe-commits. Replace a string append operation in addFunctionMappingRecord with a vector append. The existing behavior is quadratic in the worst case: this patch makes it linear. http://reviews.llvm.org/D16395 File