[PATCH] D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions

Danilo Carvalho Grael via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 15 14:06:34 PST 2020


dancgr marked an inline comment as done.
dancgr added a comment.

In D72799#1822788 <https://reviews.llvm.org/D72799#1822788>, @efriedma wrote:

> In D72799#1822599 <https://reviews.llvm.org/D72799#1822599>, @dancgr wrote:
>
> > In D72799#1822588 <https://reviews.llvm.org/D72799#1822588>, @efriedma wrote:
> >
> > > Can we also also add mul patterns for targets that have SVE, but not SVE2?
> >
> >
> > That instruction is restricted to SVE2. Do we have unpredicated vector mul instructions for SVE targets as well?
>
>
> We can synthesize a predicate using ptrue.  We do this in a few places currently; for example, to implement bswap.


I had't thought of that. I will implement SVE fallbacks for the SVE2 instructions using ptrue.



================
Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td:1555
 
 def int_aarch64_sve_and_z   : AdvSIMD_Pred2VectorArg_Intrinsic;
 def int_aarch64_sve_bic_z   : AdvSIMD_Pred2VectorArg_Intrinsic;
----------------
efriedma wrote:
> dancgr wrote:
> > @efriedma Here!
> I think here the "_z" is meant to indicate that the predicate operand is "<Pg>/Z".
Ok, I will update it to _m to follow the same pattern.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72799





More information about the llvm-commits mailing list