[all-commits] [llvm/llvm-project] dddd59: [CSSPGO][llvm-profgen] Fix getCanonicalFnName usag...

ictwanglei via All-commits all-commits at lists.llvm.org
Mon Mar 15 21:01:44 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dddd590fd0bcc7e3f0cc9769fe3bad8556aa5c8a
      https://github.com/llvm/llvm-project/commit/dddd590fd0bcc7e3f0cc9769fe3bad8556aa5c8a
  Author: wlei <wlei at fb.com>
  Date:   2021-03-15 (Mon, 15 Mar 2021)

  Changed paths:
    A llvm/test/tools/llvm-profgen/Inputs/unique-linkage-name-dwarf.perfbin
    A llvm/test/tools/llvm-profgen/Inputs/unique-linkage-name-dwarf.perfscript
    A llvm/test/tools/llvm-profgen/Inputs/unique-linkage-name-probe.perfbin
    A llvm/test/tools/llvm-profgen/Inputs/unique-linkage-name-probe.perfscript
    A llvm/test/tools/llvm-profgen/fname-canonicalization.test
    M llvm/test/tools/llvm-profgen/symbolize.ll
    M llvm/tools/llvm-profgen/ProfiledBinary.cpp
    M llvm/tools/llvm-profgen/PseudoProbe.cpp

  Log Message:
  -----------
  [CSSPGO][llvm-profgen] Fix getCanonicalFnName usage in llvm-profgen

Previously we didn't support to keep the unique linkage name(-funique-internal-linkage-name) in llvm-profgen. As discussed in https://reviews.llvm.org/D96932, we choose to do canonicalization for it.

Now since "selected" is set as the default parameter of getCanonicalFnName in `D96932`, we don't need to add any attribute here for the previous usage and only fix the missing usage in the pseudo probe decoding.

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




More information about the All-commits mailing list