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

[Simon Moll via Phabricator via llvm-commits]

simoll 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:
> 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 ;-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104673