[all-commits] [llvm/llvm-project] b5af66: [BOLT] Map branch source address to the containing...

[Amir Ayupov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b5af667b01458e9083256f2614df175916c73e5a
      https://github.com/llvm/llvm-project/commit/b5af667b01458e9083256f2614df175916c73e5a
  Author: Amir Ayupov <aaupov at fb.com>
  Date:   2024-05-12 (Sun, 12 May 2024)

  Changed paths:
    M bolt/lib/Profile/DataAggregator.cpp
    A bolt/test/X86/Inputs/blarge_new_bat_branchentry.preagg.txt
    M bolt/test/X86/bolt-address-translation-yaml.test

  Log Message:
  -----------
  [BOLT] Map branch source address to the containing basic block in BAT YAML

Fix an issue where the profile for all branches that have a BRANCHENTRY
is dropped. If the branch has an entry in BAT, it will be translated to
its input offset. We used to only permit the basic block offset as a
branch source. Perform a lookup of containing basic block instead.

Test Plan: Updated bolt-address-translation-yaml.test

Reviewers: maksfb, dcci, rafaelauler, ayermolo

Reviewed By: maksfb

Pull Request: https://github.com/llvm/llvm-project/pull/91273



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