[PATCH] D71970: [profile] Support merge pool size >= 10

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 6 12:05:24 PST 2020


MaskRay added a comment.

In D71970#1806455 <https://reviews.llvm.org/D71970#1806455>, @vsk wrote:

> What kind of performance improvement were you able to measure for your workload with n >= 10? What was the impact on profile size?


It has been a while since I measured the difference. If I use `LLVM_PROFILE_FILE=...%4m` for llc and `LLVM_PROFILE_FILE=/dev/null` for other tools, `%4m` vs `%9m` has significant performance improvement testing `test/CodeGen/X86`, maybe 30% (I may misremember the number) or something. `%9m` vs `%20m` has insignificant difference. If my machine has more cores or if I choose a better benchmark, I may see a difference :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71970/new/

https://reviews.llvm.org/D71970





More information about the llvm-commits mailing list