[all-commits] [llvm/llvm-project] fd9604: [BOLT] Set valid index for functions with profiles

Vladislav Khmelevsky via All-commits all-commits at lists.llvm.org
Wed Jun 8 04:14:55 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fd9604952d80d62bc3db57fff07c047bb6773903
      https://github.com/llvm/llvm-project/commit/fd9604952d80d62bc3db57fff07c047bb6773903
  Author: Vladislav Khmelevsky <och95 at yandex.ru>
  Date:   2022-06-08 (Wed, 08 Jun 2022)

  Changed paths:
    M bolt/lib/Passes/BinaryPasses.cpp
    M bolt/lib/Passes/ReorderFunctions.cpp

  Log Message:
  -----------
  [BOLT] Set valid index for functions with profiles

Some of the passes that calculates tentative layout like LongJmp and
Golang are expecting that only functions with valid index will be
located in hot text section. But currently functions with valid profiles
and not set index are breaking this logic, to fix this we can move the
hasValidProfile() condition from AssignSections pass to ReorderFunctions.

Vladislav Khmelevsky,
Advanced Software Technology Lab, Huawei

Differential Revision: https://reviews.llvm.org/D127223




More information about the All-commits mailing list