Re: [PATCH, RS6000 PR target/94954] Fix wrong codegen for vec_pack_to_short_fp32() builtin

2020-06-15 Thread will schmidt via Gcc-patches
On Fri, 2020-06-12 at 18:31 -0500, Segher Boessenkool wrote: > Hi! > > On Thu, Jun 11, 2020 at 11:22:33PM -0500, will schmidt wrote: > > Fix codegen implementation for the builtin > > vec_pack_to_short_fp32. > > +;; Convert two vector F32 to packed vector f16. > > +(define_expand "convert_4f32_8

Re: [PATCH, RS6000 PR target/94954] Fix wrong codegen for vec_pack_to_short_fp32() builtin

2020-06-12 Thread Segher Boessenkool
Hi! On Thu, Jun 11, 2020 at 11:22:33PM -0500, will schmidt wrote: > Fix codegen implementation for the builtin vec_pack_to_short_fp32. > +;; Convert two vector F32 to packed vector f16. > +(define_expand "convert_4f32_8f16" > + [(set (match_operand:V8HI 0 "register_operand" "=v") > + (unsp

Re: [PATCH, RS6000 PR target/94954] Fix wrong codegen for vec_pack_to_short_fp32() builtin

2020-06-12 Thread will schmidt via Gcc-patches
On Fri, 2020-06-12 at 10:24 -0400, David Edelsohn wrote: > Hi, Will Hi, > > On Fri, Jun 12, 2020 at 12:22 AM will schmidt < > will_schm...@vnet.ibm.com> wrote: > > > > > > Hi, > > Fix codegen implementation for the builtin > > vec_pack_to_short_fp32. > > > > Regtests are underway agains

Re: [PATCH, RS6000 PR target/94954] Fix wrong codegen for vec_pack_to_short_fp32() builtin

2020-06-12 Thread David Edelsohn via Gcc-patches
Hi, Will On Fri, Jun 12, 2020 at 12:22 AM will schmidt wrote: > > > Hi, > Fix codegen implementation for the builtin vec_pack_to_short_fp32. > > Regtests are underway against powerpc64 (power7be,power8le,power9le). > (this is a power9 builtin, so should be a noop for most of those). > OK

[PATCH, RS6000 PR target/94954] Fix wrong codegen for vec_pack_to_short_fp32() builtin

2020-06-11 Thread will schmidt via Gcc-patches
Hi, Fix codegen implementation for the builtin vec_pack_to_short_fp32. Regtests are underway against powerpc64 (power7be,power8le,power9le). (this is a power9 builtin, so should be a noop for most of those). OK for trunk and backports? Thanks -Will [gcc] targ