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

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer 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.
----------------
gilr wrote:
> 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.
Ah yeah, I do see that now.
Thanks


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

https://reviews.llvm.org/D93615