minglotus-6 wrote:

> The invoke instruction can have 3 different kinds of prof data
> 
> 1. call count (if a direct call)
> 2. VP profile data (if an indirect call)
> 3. branch weights for landing pad.
> 4. can coexist with  2) and does not need to be updated. Is there an existing 
> test coverage for type 1) update?

Added test cases for 1 and 2 in the pre-commit 
[PR](https://github.com/llvm/llvm-project/pull/83780) so the diff in this one 
is clearer.

As discussed offline, non-count prof data (i.e., representing taken or not 
taken branches, associated with `br` or `switchinst`, etc) doesn't need scaling 
so no extra work needed. PTAL.


https://github.com/llvm/llvm-project/pull/83809
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to