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

[Satish K Guggilla via Phabricator via llvm-commits]

sguggill added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:1404
-  // loop of every loop nest.
-  if (L.empty() || VPlanBuildStressTest ||
-      (EnableVPlanNativePath && isExplicitVecOuterLoop(&L, ORE))) {
----------------
The intent here is to stress test VPlan build for any outer loop that is reducible (even ones that are not marked with clang vectorize enable). We lose that with this change and I don't think that is the right thing. Given this I think, we will need the VPlanBuildStressTest flag.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59952