[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