[PATCH] D109036: [CSSPGO] Sort function offset table to speed up profile loading.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 1 12:07:49 PDT 2021


hoy added inline comments.


================
Comment at: llvm/lib/ProfileData/SampleProfReader.cpp:791
+      for (auto &NameOffset : *OrderedFuncOffsets) {
+        auto &FContext = NameOffset.first;
+        auto FName = FContext.getName();
----------------
wmi wrote:
> can it be a const?
Yes, const should work. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109036



More information about the llvm-commits mailing list