This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] [AArch64, NEON] Improve vpmaxX & vpminX intrinsics


> You'll need to rebase over Alan Lawrance's patch.
> https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00279.html

Yes, see my new patch: https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00750.html 

> > +;; Pairwise Integer Max/Min operations.
> > +(define_insn "aarch64_<maxmin_uns>p<mode>"
> > + [(set (match_operand:VQ_S 0 "register_operand" "=w")
> > +       (unspec:VQ_S [(match_operand:VQ_S 1 "register_operand" "w")
> > +                    (match_operand:VQ_S 2 "register_operand" "w")]
> > +                   MAXMINV))]
> > + "TARGET_SIMD"
> > + "<maxmin_uns_op>p\t%0.<Vtype>, %1.<Vtype>, %2.<Vtype>"
> > +  [(set_attr "type" "neon_minmax<q>")]
> > +)
> > +
> 
> Could you roll aarch64_reduc_<maxmin_uns>_internalv2si into this pattern?

Will come up with another patch to fix this.  Thanks for pointing this out.

> 
> Thanks,
> Tejas.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]