[PATCH] D69891: [VP,Integer,#1] Vector-predicated integer intrinsics

Fri Mar 13 02:58:47 PDT 2020

fhahn added a comment.

In D69891#1921124 <https://reviews.llvm.org/D69891#1921124>, @simoll wrote:

> In D69891#1919830 <https://reviews.llvm.org/D69891#1919830>, @andrew.w.kaylor wrote:
>
> > OK. Since the behavior for out-of-range evl is target-dependent, undefined makes sense.
> >
> > I don't know if you were waiting for input from anyone else, but this looks good to me.
>
>
> Great :) Thanks to everybody involved for reviewing!
>  We've iterated on this patch for a while and the last major update was weeks ago, giving people enough time to react to it. I'll commit this today.

Given the scale of the change, it might be good to let people on llvm-dev know that a this patch is ready to land and it is about to be committed (and wait till early next week in case there's additional feedback).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69891