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

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 16 14:27:06 PDT 2019


hsaito added a comment.

In D57504#1468510 <https://reviews.llvm.org/D57504#1468510>, @simoll wrote:

>   1. Updates
> - added constrained fp intrinsics (IR level only).
> - initial support for mapping `llvm.experimental.constrained.*` intrinsics to `llvm.vp.constrained.*`.


Do we really need both vp.fadd() and vp.constrained.fadd()? Can't we just use the latter with rmInvalid/ebInvalid? That should prevent vp.constrained.fadd from losing optimizations w/o good reasons.
Do we have enough upside in having both?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57504





More information about the llvm-commits mailing list