[PATCH] D157462: LoopRotate: Add code to update branch weights

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 16:56:08 PDT 2023


MatzeB added inline comments.


================
Comment at: llvm/include/llvm/IR/ProfDataUtils.h:67-71
+/// Faster version of extractBranchWeights() that skips checks and must only
+/// be called with "branch_weights" metadata nodes.
+void extractFromBranchWeightMD(const MDNode *ProfileData,
+                               SmallVectorImpl<uint32_t> &Weights);
+
----------------
paulkirth wrote:
> Would you mind putting the changes to ProfDataUtils in a separate patch? I think these are good additions to the library, so I think it's better if they go in their own commit. Since it just renames an internal interface(practically speaking), it can probably be marked NFC/NFCI.
Extracted as D157937 now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157462



More information about the llvm-commits mailing list