[all-commits] [llvm/llvm-project] 865260: [BOLT] Fix counts aggregation in merge-fdata (#119...

Amir Ayupov via All-commits all-commits at lists.llvm.org
Sat Dec 14 22:38:45 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 86526084044167b3c753d32ef8dbf79d57cba0c4
      https://github.com/llvm/llvm-project/commit/86526084044167b3c753d32ef8dbf79d57cba0c4
  Author: Amir Ayupov <aaupov at fb.com>
  Date:   2024-12-14 (Sat, 14 Dec 2024)

  Changed paths:
    M bolt/test/merge-fdata-lbr-mode.test
    M bolt/tools/merge-fdata/merge-fdata.cpp

  Log Message:
  -----------
  [BOLT] Fix counts aggregation in merge-fdata (#119652)

merge-fdata used to consider misprediction count as part of "signature",
or the aggregation key. This prevented it from collapsing profile lines
with different misprediction counts, which resulted in duplicate
`(from, to)` pairs with different misprediction and execution counts.

Fix that by splitting out misprediction count and accumulating it
separately.

Test Plan: updated bolt/test/merge-fdata-lbr-mode.test



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


More information about the All-commits mailing list