[all-commits] [llvm/llvm-project] cbd302: [BOLT] Improve BinaryFunction::inferFallThroughCou...

[ShatianWang via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cbd302410e9f27013223a96edcd78dfb597979e1
      https://github.com/llvm/llvm-project/commit/cbd302410e9f27013223a96edcd78dfb597979e1
  Author: ShatianWang <38512325+ShatianWang at users.noreply.github.com>
  Date:   2024-08-21 (Wed, 21 Aug 2024)

  Changed paths:
    M bolt/lib/Core/BinaryFunctionProfile.cpp
    A bolt/test/X86/infer-fall-throughs.s

  Log Message:
  -----------
  [BOLT] Improve BinaryFunction::inferFallThroughCounts() (#105450)

This PR improves how basic block execution count is updated when using
the BOLT option `-infer-fall-throughs`. Previously, if a 0-count
fall-through edge is assigned a positive inferred count N, then the
successor block's execution count will be incremented by N. Since the
successor's execution count is calculated using information besides
inflow sum (such as outflow sum), it likely is already correct, and
incrementing it by an additional N would be wrong. This PR improves how
the successor's execution count is updated by using the max over its
current count and N.



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