[PATCH] D99750: [LV, VP] RFC: VP intrinsics support for the Loop Vectorizer (Proof-of-Concept)

Thu May 18 07:03:05 PDT 2023

ABataev added a comment.

In D99750#4350780 <https://reviews.llvm.org/D99750#4350780>, @fhahn wrote:

> It looks like the description is out-of date, it would be great if it could be updated to be in sync with the patch. @rogfer01  and @loralb had a poster at EuroLLVM about a different approach and I am curious if you by any chance already took a look at their approach?

Checked the poster, actually it is pretty similar to the proposed patch, and, most probably, can be reused by their approach too. @rogfer01?

================
Comment at: llvm/test/Transforms/LoopVectorize/X86/vectorize-vp-intrinsics.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt -passes=loop-vectorize \
----------------
fhahn wrote:
> Do the current changes have any impact on the test?
It checks that the new options do not break anything for other targets and non-scalable vectorization. 

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99750