[PATCH] D57504: RFC: Prototype & Roadmap for vector predication in LLVM

Tue Dec 3 10:06:18 PST 2019

rkruppe added inline comments.

================
Comment at: llvm/docs/LangRef.rst:15283-15300
+.. _int_vp_select:
+
+'``llvm.vp.select.*``' Intrinsics
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+Syntax:
+"""""""
----------------
simoll wrote:
> > @rkruppe 
> > [..] My main point was just that the existing select instruction is not sufficient as the second operation, for essentially the same reason why the VP intrinsics have an EVL argument instead of just the mask. Creating a VP equivalent of select (as already sketched in the other thread) resolves that concern just as well.
> I agree. The prototype has defined such an `llvm.vp.select` from the get-go.
Oops, missed that / forgot about it. Sorry for the noise.

Is there a reason why it's not in the "integer slice" patch? It's not integer-specific, but it seems to fit even less into the other slices.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57504