[all-commits] [llvm/llvm-project] 3f78d8: [TypeProf][InstrFDO]Omit vtable symbols in indexed...

[Mingming Liu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3f78d89a2e6170d206a6b91a93b3fdf5e46ab6db
      https://github.com/llvm/llvm-project/commit/3f78d89a2e6170d206a6b91a93b3fdf5e46ab6db
  Author: Mingming Liu <mingmingl at google.com>
  Date:   2024-06-26 (Wed, 26 Jun 2024)

  Changed paths:
    M compiler-rt/test/profile/Linux/instrprof-vtable-value-prof.cpp
    M llvm/test/tools/llvm-profdata/vtable-value-prof.test
    M llvm/tools/llvm-profdata/llvm-profdata.cpp

  Log Message:
  -----------
  [TypeProf][InstrFDO]Omit vtable symbols in indexed profiles by default (#96520)

- The indexed iFDO profiles contains compressed vtable names for `llvm-profdata show --show-vtables` debugging 
   usage. An optimized build doesn't need it and doesn't decompress the blob now [1], since optimized binary has the 
   source code and IR to find vtable symbols.
- The motivation is to avoid increasing profile size when it's not necessary.
- This doesn't change the indexed profile format and thereby doesn't need a version change.

[1] https://github.com/llvm/llvm-project/blob/eac925fb81f26342811ad1765e8f9919628e2254/llvm/include/llvm/ProfileData/InstrProfReader.h#L696-L699



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications