[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
Tue Aug 31 23:00:25 PDT 2021
hoy added inline comments.
================
Comment at: llvm/lib/ProfileData/SampleProfWriter.cpp:44
+static cl::opt<bool> SortFuncOffsetTable(
+ "sort-func-offsets", cl::Hidden, cl::init(true),
----------------
wenlei wrote:
> hoy wrote:
> > wenlei wrote:
> > > Why do we need a flag? Can we simplify this by always order the function offset table for CSSPGO profile?
> > This is mostly for debugging and verification of the unordered path that is kept for being compatible with legacy unordered profiles.
> I don't think we have a backward-compatibility issue, there's no need to keep the legacy unordered profile. Can we remove it? Testing just need to make sure this ordering change does not affecting importing.
Makes sense. Removed the switch.
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