[PATCH] D102522: [llvm-exegesis] Loop unrolling for loop snippet repetitor mode

[Clement Courbet via Phabricator via llvm-commits]

courbet accepted this revision.
courbet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/tools/llvm-exegesis/lib/BenchmarkRunner.cpp:172
+    const int kMinInstructionsForSnippet = 4 * Instructions.size();
+    const int kLoopBodySizeForSnippet = 2 * Instructions.size();
     {
----------------
lebedev.ri wrote:
> courbet wrote:
> > ditto
> So if i pass `LoopBodySize` into constructor of `LoopSnippetRepetitor`,
> how can i adjust it here then?
Right, I don't have a good suggestion... Let's keep it like this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102522