[PATCH] D59952: [VPLAN] Remove option for stress testing.

[Hideki Saito via Phabricator via llvm-commits]

hsaito added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:267
 
-// This flag enables the stress testing of the VPlan H-CFG construction in the
-// VPlan-native vectorization path. It must be used in conjuction with
----------------
fhahn wrote:
> Ok I think we still want to keep this one here, as it enables building VPlans for loops without the pragma. But we can remove the code to arbitrarily pick VF = 4.
We need VF>1 to make this meaningful, and I don't think we'll get that from the current VF determination code. I'm fine setting VF=4 only when the computed VF is 1. Blindly setting VF to a fixed value gives more determinism, though.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59952