[all-commits] [llvm/llvm-project] 51d382: [ThinLTO] Shrink FunctionSummary by 8 bytes (#107706)

[Kazu Hirata via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 51d3829d8fc5beba269629903365af75174de7f6
      https://github.com/llvm/llvm-project/commit/51d3829d8fc5beba269629903365af75174de7f6
  Author: Kazu Hirata <kazu at google.com>
  Date:   2024-09-07 (Sat, 07 Sep 2024)

  Changed paths:
    M llvm/include/llvm/AsmParser/LLParser.h
    M llvm/include/llvm/IR/ModuleSummaryIndex.h
    M llvm/include/llvm/IR/ModuleSummaryIndexYAML.h
    M llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
    M llvm/lib/AsmParser/LLParser.cpp
    M llvm/lib/Bitcode/Reader/BitcodeReader.cpp
    M llvm/lib/IR/ModuleSummaryIndex.cpp

  Log Message:
  -----------
  [ThinLTO] Shrink FunctionSummary by 8 bytes (#107706)

During the ThinLTO indexing step for one of our large applications, we
create 4 million instances of FunctionSummary.

Changing:

  std::vector<EdgeTy> CallGraphEdgeList;

to:

  SmallVector<EdgeTy, 0> CallGraphEdgeList;

in FunctionSummary reduces the size of each instance by 8 bytes.  The
rest of the patch makes the same change to other places so that the
types stay compatible across function boundaries.



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