[PATCH] D156715: [CSSPGO] Fix issues with post-link function checksum check

Fri Aug 4 14:45:54 PDT 2023

hoy added a comment.

In D156715#4558585 <https://reviews.llvm.org/D156715#4558585>, @wlei wrote:

> In D156715#4550679 <https://reviews.llvm.org/D156715#4550679>, @hoy wrote:
>
>> How does perf look like after disabling annotating imported functions?
>
> So far from what I tested, there is no perf change if only disabling annotating imported functions. Note that this also disables the profile matching for the  imported functions, so it could cause perf issue, depending on if the imported functions is stale.

Discussed offline. This effect of this change is potentially two-fold :

1. positive impact of disabling unnecessary matching for non-stale profiles
2. negative impact of disabling matching for to-be-inlined imported functions with a stale profile.

Currently they are not tested separately. With a solution of #1, allowing matching for #2  might be a win. Suggesting to hold on this change until we have a good solution for #1.

Thoughts?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156715