[PATCH] D98634: [LV] Move runtime pointer size check to LVP::plan().

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

So the basic idea here is that LVL should only care about legality, but it should be up to it's users to care about profitability?
What do you think about the other limit?



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp:45-52
 static cl::opt<unsigned> VectorizeSCEVCheckThreshold(
     "vectorize-scev-check-threshold", cl::init(16), cl::Hidden,
     cl::desc("The maximum number of SCEV checks allowed."));
 
 static cl::opt<unsigned> PragmaVectorizeSCEVCheckThreshold(
     "pragma-vectorize-scev-check-threshold", cl::init(128), cl::Hidden,
     cl::desc("The maximum number of SCEV checks allowed with a "
----------------
These should move, too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98634