[MemProf] Fix combined index handling for locals
authorTeresa Johnson <tejohnson@google.com>
Fri, 30 Dec 2022 15:47:12 +0000 (07:47 -0800)
committerTeresa Johnson <tejohnson@google.com>
Mon, 9 Jan 2023 22:11:43 +0000 (14:11 -0800)
commitb1b9f1789e63843a602a30ef5b0a0bf22ef9e12f
tree5a9c177ae836bbe9cfc8e5a405b0ffa2369667ea
parent5b72d0e4f5eeb8f90c744cac8e0728cffeca61a9
[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
llvm/lib/Bitcode/Reader/BitcodeReader.cpp
llvm/test/ThinLTO/X86/memprof-summary.ll