[all-commits] [llvm/llvm-project] 594398: [PSI] Ensure hot and could counter thresholds are ...

[xur-llvm via All-commits]

  Branch: refs/heads/users/xur-llvm/icp_work
  Home:   https://github.com/llvm/llvm-project
  Commit: 594398aa35f2c95beddd296673f38a3308606d90
      https://github.com/llvm/llvm-project/commit/594398aa35f2c95beddd296673f38a3308606d90
  Author: Rong Xu <xur at google.com>
  Date:   2025-07-21 (Mon, 21 Jul 2025)

  Changed paths:
    M llvm/lib/Analysis/ProfileSummaryInfo.cpp

  Log Message:
  -----------
  [PSI] Ensure hot and could counter thresholds are distinct

If hot and cold thresholds are identical, both isColdCount() and
isHotCount() can return true due to inclusive checks in these
functions. This is undesirable.

This patch ensures distinct hot and cold counter thresholds.
An alternative fix is to make the check in isColdCount() exclusive.
But that requires many test case changes.


  Commit: 73bea24b0c4591e9aaefd318223ae67768740c4c
      https://github.com/llvm/llvm-project/commit/73bea24b0c4591e9aaefd318223ae67768740c4c
  Author: Rong Xu <xur at google.com>
  Date:   2025-07-21 (Mon, 21 Jul 2025)

  Changed paths:
    M llvm/lib/Transforms/Instrumentation/IndirectCallPromotion.cpp
    M llvm/test/ThinLTO/X86/memprof-icp.ll

  Log Message:
  -----------
  [ICP] Add a few tunings to indirect-call-promtion

Indirect-call promotion (ICP) has been adjusted with the following
tunings, both of which are enabled by default:
  (1) Candidate functions can be now ICP'd even if only a declaration
      is present.
  (2) All non-cold candidate functions are now considered by ICP.
      Previously, only hot targets were considered.
These changes are expected to improve performance, with some large
Google benchmarks showing up to a 0.3% improvement.


Compare: https://github.com/llvm/llvm-project/compare/594398aa35f2%5E...73bea24b0c45

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications