[PATCH] D90880: [LoopVectorizer] NFC: Return ElementCount from compute[Feasible]MaxVF

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Is expected to make anything work yet for scalable vectors, or is it just updating interfaces? (i.e. will the code be checked somehow to make sure it will work for scalable vectors).



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5275
+      UserIC ? MaxVF.getKnownMinValue() * UserIC : MaxVF.getKnownMinValue();
   if (TC > 0 && TC % MaxVFtimesIC == 0) {
     // Accept MaxVF if we do not have a tail.
----------------
Would this TC mod check still be valid for scalable vectors? If not does it need an assert?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90880