[all-commits] [llvm/llvm-project] d55e23: [memprof] Use std::unique_ptr instead of std::opti...

[Kazu Hirata via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d55e235b2384281a5d1d982094fb2f819999885b
      https://github.com/llvm/llvm-project/commit/d55e235b2384281a5d1d982094fb2f819999885b
  Author: Kazu Hirata <kazu at google.com>
  Date:   2024-06-06 (Thu, 06 Jun 2024)

  Changed paths:
    M llvm/include/llvm/ProfileData/MemProf.h
    M llvm/lib/ProfileData/MemProfReader.cpp

  Log Message:
  -----------
  [memprof] Use std::unique_ptr instead of std::optional (#94655)

Changing the type of Frame::SymbolName from std::optional<std::string>
to std::unique<std::string> reduces sizeof(Frame) from 64 to 32.

The smaller type reduces the cycle and instruction counts by 23% and
4.4%, respectively, with "llvm-profdata show" modified to deserialize
all MemProfRecords in a MemProf V2 profile.  The peak memory usage is
cut down nearly by half.



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