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

Rodrigo Caetano Rocha via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 28 16:57:04 PDT 2019


rcorcs added a comment.

Just a very minor thing.



================
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,
----------------
Maybe it is better to call this variable Look*A*headOperandsBudget, similar to the other ones.


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