[PATCH] D114319: [InstrProf][NFC] Save profile bias to function map
Petr Hosek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 13 11:20:56 PDT 2022
phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: StephenFan.
Herald added a project: All.
LGTM
@ellis do you have time to land this change? This change should address https://github.com/llvm/llvm-project/issues/55125 so I'd like to see it landed.
================
Comment at: llvm/include/llvm/Transforms/Instrumentation/InstrProfiling.h:62
+ /// instruction that produces the profile relocation bias.
+ DenseMap<const Function *, LoadInst *> FunctionToProfileBiasLI;
std::vector<GlobalValue *> CompilerUsedVars;
----------------
I'd prefer `FunctionToProfileBiasMap` to be consistent with other variables like `ProfileDataMap`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114319/new/
https://reviews.llvm.org/D114319
More information about the llvm-commits
mailing list