[PATCH] D63948: [SLP] Limit compilation time of look-ahead operand reordering heuristic.

Vasileios Porpodas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 28 17:02:36 PDT 2019


vporpo marked an inline comment as done.
vporpo added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:158
+// increase we limit the number of operands visited to this value.
+static cl::opt<unsigned> LookaheadOperandsBudget(
+    "slp-look-ahead-operands-budget", cl::init(2), cl::Hidden,
----------------
rcorcs wrote:
> Maybe it is better to call this variable Look*A*headOperandsBudget, similar to the other ones.
Yes, good point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63948





More information about the llvm-commits mailing list