[PATCH] D41152: Use a custom container to reduce AFDO's memory usage by ~60%

George Burgess IV via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 14 15:34:56 PST 2017


george.burgess.iv abandoned this revision.
george.burgess.iv added a comment.

I like that idea. :)

Also, it looks like I was mistaken: `CallTargetMap` is better off a `StringMap` than a `std::map<std::string, uint64_t>`. The difference was around 5%, which makes step one save us 17% of memory. Checked it in as r320764, as requested.

I'll do step 2 and step 3 in fresh, more targeted reviews.

Thanks again!


https://reviews.llvm.org/D41152





More information about the llvm-commits mailing list