[all-commits] [llvm/llvm-project] c8365f: [ctx_prof] Simple ICP criteria during module inlin...

[Mircea Trofin via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c8365feed7af6d17cd2cc04cdc7fe0247f87e9c8
      https://github.com/llvm/llvm-project/commit/c8365feed7af6d17cd2cc04cdc7fe0247f87e9c8
  Author: Mircea Trofin <mtrofin at google.com>
  Date:   2024-09-25 (Wed, 25 Sep 2024)

  Changed paths:
    M llvm/include/llvm/Analysis/CtxProfAnalysis.h
    M llvm/lib/Analysis/CtxProfAnalysis.cpp
    M llvm/lib/Transforms/IPO/ModuleInliner.cpp
    M llvm/lib/Transforms/Utils/CallPromotionUtils.cpp
    A llvm/test/Analysis/CtxProfAnalysis/flatten-icp.ll

  Log Message:
  -----------
  [ctx_prof] Simple ICP criteria during module inliner (#109881)

This is mostly for test: under contextual profiling, we perform ICP for those indirect callsites which have targets marked as `alwaysinline`.

This helped uncover a bug with the way the profile was updated upon ICP, where we were skipping over the update if the target wasn't called in that context. That was resulting in incorrect counts for the indirect BB.

Also flyby fix to the total/direct count values, they should be 64-bit (as all counters are in the contextual profile)



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