[PATCH] D91230: [IRCE] Increase min runtime iteration threshold

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 10 23:24:03 PST 2020


skatkov added a comment.

In D91230#2387803 <https://reviews.llvm.org/D91230#2387803>, @mkazantsev wrote:

> It looks like the threshold 10 you are referring is coming from some option. Can we use the same option in both places instead?

you mean to combine 
static cl::opt<int> MaxExitProbReciprocal("irce-max-exit-prob-reciprocal",

  cl::Hidden, cl::init(10));

and
static cl::opt<unsigned> MinRuntimeIterations("min-runtime-iterations",

  cl::Hidden, cl::init(10));

into one option?
Are you ok with the name min-runtime-iterations?


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

https://reviews.llvm.org/D91230



More information about the llvm-commits mailing list