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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 27 14:11:11 PDT 2019


fhahn added a comment.

Thanks Francesco! I'll commit the change tomorrow, unless @hsaito does it today :)



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6103
+      // We set VF to 4 for stress testing.
+      if (VPlanBuildStressTest)
+        VF = 4;
----------------
fpetrogalli wrote:
> fhahn wrote:
> > I think as a follow up, we can drop VPlanBuildStressTest, now that we do not require a UserVF to build VPlans.
> Should I address this in a separate patch or this one?
Yep that would be great!


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

https://reviews.llvm.org/D57598





More information about the llvm-commits mailing list