[PATCH] D59952: [VPLAN] Use computed VF for stress testing.

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli updated this revision to Diff 192713.
fpetrogalli retitled this revision from "[VPLAN] Remove option for stress testing." to "[VPLAN] Use computed VF for stress testing.".
fpetrogalli edited the summary of this revision.
Herald added a subscriber: jdoerfert.

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

https://reviews.llvm.org/D59952

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp


Index: lib/Transforms/Vectorize/LoopVectorize.cpp
===================================================================
--- lib/Transforms/Vectorize/LoopVectorize.cpp
+++ lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -6099,13 +6099,12 @@
     // If the user doesn't provide a vectorization factor, determine a
     // reasonable one.
     if (!UserVF) {
-      // We set VF to 4 for stress testing.
-      if (VPlanBuildStressTest)
-        VF = 4;
-      else
         VF = determineVPlanVF(TTI->getRegisterBitWidth(true /* Vector*/), CM);
-    }
 
+        // Make sure we have a VF > 1 for stress testing.
+        if (VPlanBuildStressTest && VF < 2)
+          VF = 4;
+    }
     assert(EnableVPlanNativePath && "VPlan-native path is not enabled.");
     assert(isPowerOf2_32(VF) && "VF needs to be a power of two");
     LLVM_DEBUG(dbgs() << "LV: Using " << (UserVF ? "user VF " : "computed VF ")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59952.192713.patch
Type: text/x-patch
Size: 901 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190328/fa3daf48/attachment.bin>