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

[Krzysztof Parzyszek via Phabricator via llvm-commits]

kparzysz 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
+
----------------
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.


Repository:
  rL LLVM

https://reviews.llvm.org/D45271