[PATCH] D102429: [CSSPGO] Update pseudo probe distribution factor based on inline context.

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 14 11:25:17 PDT 2021


wlei added a comment.

Try to understand this:

So we merge the samples from the same function but different context, does this cause the issue?

> Such probes should not share samples and their factors should be fixed up separately.

So with that, when code duplication happens, we need to fix probe factor, so here it will do it separately by grouping the probe by different inline context. then each group work for one context inlining. is my understanding correct?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102429



More information about the llvm-commits mailing list