[all-commits] [llvm/llvm-project] 4ecf2c: [BOLT] Use aggregated FuncBranchData in writeBATYAML

Amir Ayupov via All-commits all-commits at lists.llvm.org
Mon May 13 14:23:57 PDT 2024


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

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

  Log Message:
  -----------
  [BOLT] Use aggregated FuncBranchData in writeBATYAML

Switch from FuncBranchData intermediate maps (Intra/InterIndex)
to aggregated Data, same as one used by DataReader:
https://github.com/llvm/llvm-project/blob/e62ce1f8842cca36eb14126d79dcca0a85bf6d36/bolt/lib/Profile/DataReader.cpp#L385-L389
This aligns the order of the output between YAMLProfileWriter and
writeBATYAML.

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

Reviewers: rafaelauler, dcci, ayermolo, maksfb

Reviewed By: ayermolo, maksfb

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



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