[PATCH] D104673: [LV, VP] Add CLI option to support predicated vectorization

[Vineet Kumar via Phabricator via llvm-commits]

vkmr added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:238
 
+// Option prefer-predicate-with-vp-intrinsics is an experimental switch to
+// indicate that the loop vectorizer should try to generate VP intrinsics if
----------------
simoll wrote:
> simoll wrote:
> > This is a lot of text. Maybe restructure this to make clear what the default is right now (NoPredication), what the default should be in the future (IfAVLSupported) and mention the other two explicitly as experimental/test settings.
> @vkmr pong: I stand by that comment ;-)
Oops my bad! @simoll I 100% stand by your comment too! 
I totally forgot about it before the ping.  I will address it in an upcoming update.





Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104673