[PATCH] D92059: [SLP] Control maximum vectorization factor from TTI

Mon Dec 7 15:13:30 PST 2020

rampitec added a comment.

In D92059#2438262 <https://reviews.llvm.org/D92059#2438262>, @jonpa wrote:

> @rampitec : It would be nice to see this go in soon...

Sorry, just returned from vacation. I believe I have addressed the comments.

================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:748

+  unsigned getMaximumVF(unsigned ElemWidth, unsigned Opcode) const {
+    unsigned MaxVF = MaxVFOption.getNumOccurrences() ?
----------------
jonpa wrote:
> Is there a reason for having the default TTI value of 0 instead of UINT_MAX directly as you first suggested?
> 
> 
Yes, there is getMinimumVF() with default 0. This is just for the uniformity.

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

https://reviews.llvm.org/D92059