[all-commits] [llvm/llvm-project] a3d027: [MemProf] Disable alloc context in combined summar...

[Teresa Johnson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a3d027f92308890c9b1ace7b8a5a7f7d69ce5f0e
      https://github.com/llvm/llvm-project/commit/a3d027f92308890c9b1ace7b8a5a7f7d69ce5f0e
  Author: Teresa Johnson <tejohnson at google.com>
  Date:   2025-05-09 (Fri, 09 May 2025)

  Changed paths:
    M llvm/include/llvm/Bitcode/LLVMBitCodes.h
    M llvm/lib/AsmParser/LLParser.cpp
    M llvm/lib/Bitcode/Reader/BitcodeAnalyzer.cpp
    M llvm/lib/Bitcode/Reader/BitcodeReader.cpp
    M llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
    M llvm/test/Assembler/thinlto-memprof-summary.ll
    M llvm/test/ThinLTO/X86/memprof_direct_recursion.ll

  Log Message:
  -----------
  [MemProf] Disable alloc context in combined summary for ndebug builds (#139161)

Since we currently only use the context information in the alloc info
summary in the LTO backend for assertion checking, there is no need to
write this into the combined summary index for distributed ThinLTO for
NDEBUG builds. Put this under a new -combined-index-memprof-context
option which is off by default for NDEBUG.

The advantage is that we save time (not having to sort in preparation
for building the radix trees), and space in the generated bitcode files.

We could also do so for the callsite info records, but those are smaller
and less expensive to prepare.



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