[all-commits] [llvm/llvm-project] b1b9f1: [MemProf] Fix combined index handling for locals

Teresa Johnson via All-commits all-commits at lists.llvm.org
Mon Jan 9 14:12:02 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b1b9f1789e63843a602a30ef5b0a0bf22ef9e12f
      https://github.com/llvm/llvm-project/commit/b1b9f1789e63843a602a30ef5b0a0bf22ef9e12f
  Author: Teresa Johnson <tejohnson at google.com>
  Date:   2023-01-09 (Mon, 09 Jan 2023)

  Changed paths:
    M llvm/lib/Bitcode/Reader/BitcodeReader.cpp
    M llvm/test/ThinLTO/X86/memprof-summary.ll

  Log Message:
  -----------
  [MemProf] Fix combined index handling for locals

Since the linker does not resolve local symbols, we cannot look up
whether they are prevailing. The prior check was blocking all locals
from getting memprof summaries in the combined index.

Modified the existing test case to contain a local. This necessitated
some other fixes as the order of summary entries changed.

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




More information about the All-commits mailing list