[PATCH] D45271: [LV] Introduce TTI::getMinimumVF

[Hideki Saito via Phabricator via llvm-commits]

hsaito added inline comments.


================
Comment at: test/Transforms/LoopVectorize/Hexagon/minimum-vf.ll:5
+; Check that TTI::getMinimumVF works.
+; CHECK: LV: Overriding calculated MaxVF(9) with target's minimum: 64
+
----------------
kparzysz wrote:
> hsaito wrote:
> > Please add a comment saying that hard coded 9 comes from the constant trip count, in case someone has to maintain the test case and/or validation later.
> > 
> I have reduced this testcase.  The MaxVF of 9 actually came from register usage calculation.  It was a coincidence that the iteration count was 9 as well.
Can't go something as simple as the equivalent of this? If that's the case why?
for(i=0;i<9;i++){
   a[i]+=1; // int add
   b[i]+=1; // char add
}



Repository:
  rL LLVM

https://reviews.llvm.org/D45271