[PATCH] D109860: profi - a flow-based profile inference algorithm: Part I (out of 3)

[Sergey Pupyrev via Phabricator via llvm-commits]

spupyrev marked an inline comment as done.
spupyrev added a comment.

> I get the test result for our search benchmark. Enabling sample-profile-use-profi has a small regression - 0.3%.  That may be related with the issues you described above.

Thanks for running those experiments, the results align with what I've seen in my benchmarks.
At this point I think we should think of the inference algorithm as a part of CSSPGO, where it provides benefits. Making the inference work with AutoFDO is an important but a separate (and potentially time-consuming) task. I'd try to address the regression in future diffs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109860