[PATCH] D127223: [BOLT] Set valid index for functions with profiles

Maksim Panchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 15:07:48 PDT 2022


maksfb accepted this revision.
maksfb added a comment.
This revision is now accepted and ready to land.

We were considering adding another level of "hotness" for basic blocks and maybe for functions. I.e. potentially we might have hot/cold/frozen split if it proves to be beneficial. Then relying on implicit assumptions about indices will become too fragile. Something to keep in mind.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127223



More information about the llvm-commits mailing list