[PATCH] D93615: [LV] Avoid needless fold tail

[Gil Rapaport via Phabricator via llvm-commits]

gilr added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5505
       UserIC ? MaxVF.getFixedValue() * UserIC : MaxVF.getFixedValue();
   if (TC > 0 && TC % MaxVFtimesIC == 0) {
     // Accept MaxVF if we do not have a tail.
----------------
SjoerdMeijer wrote:
> Was surprised to see this change, because I thought we were handling it here already.
> Is this check here still relevant? Or can we "merge" this with the one that you added below?
>Is this check here still relevant?

Yes, since IC may take non-power-of-2 values. Will add a test to cover that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93615