[PATCH] D57598: [VPLAN] Determine Vector Width programmatically.

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli marked an inline comment as done.
fpetrogalli added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6105
+    // We set VF to 4 for stress testing.
+    if (VPlanBuildStressTest)
+      VF = 4;
----------------
hsaito wrote:
> (VPlanBuildStressTest && VF < 2) ?
> 
> 
> 
Sure, but why? If we are in `VPlanBuildStressTest`, why would you care about the value of VF?

Isn't it better to have full control on the stress tests and make sure that we always vectorize with VF = 4?


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

https://reviews.llvm.org/D57598