[PATCH] D152490: [llvm-profdata] Use StringRef for CallTargetMap

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 18:26:09 PDT 2023


hoy added a subscriber: marksantaniello.
hoy added a comment.

Interestingly we've seen regression on memory usage with similar change. The reason is that an empty DenseMap preallocates a chunk of memory that turns out more expansive than a single string for our workload. cc @marksantaniello  @wenlei


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152490/new/

https://reviews.llvm.org/D152490



More information about the llvm-commits mailing list